-
Notifications
You must be signed in to change notification settings - Fork 4.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Release a new version of gson-extras #2807
Comments
Related / similar previous issues (also give some explanation why it was not published yet):
Maybe blocked by #2042
Probably due to this in Line 37 in 528fd31
https://mvnrepository.com (which is not the official Maven Central web interface) also lists third-party repositories. It seems gson-extras was deployed to the repository of CronApp (by them probably), but not to Maven Central. So if we were to release gson-extras to Maven Central it would be the first public release. Maybe we should first go through the API and implementation to make sure there are no obvious issues (e.g. leaking implementation details in API, ...) because afterwards it might be difficult to fix this without breaking backward compatibility. Technically even now it could break backward compatibility in case users have copied the gson-extras code to their projects, but maybe the risk for that is lower (and in the worst case they could keep using their local copy). |
I guess I should have researched this a bit more carefully. It's just as well my attempt to deploy failed. :-) |
I have been operating under the assumption that the
extras
project had no Maven Central artifacts, but that's not true! We havecom.google.code.gson:gson-extras
, last released in 2018. Not much has happened since, but there is at least a change toRuntimeTypeAdapterFactory
to give it arecognizeSubtypes()
method. The existing artifact also has a CVE logged against it, and even though that is actually not a security problem at all, it would be nicer for it not to be there.I tried releasing a new version with the existing configuration, but it didn't work. It got as far as making a git tag and everything, but then said:
I'm sure there's some refinement needed to
extras/pom.xml
, probably similar to what's ingson/pom.xml
.We should also correct this statement in the README:
(It's technically correct, I suppose, since it says that "are currently not deployed", not that they never have been deployed.)
The text was updated successfully, but these errors were encountered: