Upgrade ion-java to 1.11.2 and remove handling of exceptions that are no longer leaked - #482
Conversation
|
Thank you @tgregg! There is one new Fuzz report at: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=66495 but it might be resolved by this as well, I'll see if it gets closed within 24-48 hours (there's some delay until failing tests are re-run). Btw, can you see OSS-Fuzz entries like that, or do you need more access? I use this dashboard: (which contains fails for all binary formats) |
|
@cowtowncoder I get permission denied at both of those links. Is there something you can do to grant me access, or do you know of a self-service process for me to follow? Note: I'm also going to onboard ion-java directly so I won't have to wait to receive reports via jackson-dataformats-binary. |
|
@tgregg I can ask Adam via comment to add, I think -- do you have gmail account to use? I think main auth is via google. |
Yes, tyagregg@ |
|
@tgregg Ok I'll see if I can do a PR for https://github.com/cowtowncoder/oss-fuzz; access defined in |
|
@tgregg Ok, merged -- I think you should now have access. |
|
Still getting access denied for now; I'll check again later in case the changes take time to propagate. |
|
Yeah, that could be. Let me know if it still won't work after 24 hours or so, I can ask what gives. |
|
@tgregg I think you have access now (I saw updates that were for notifying you of open issues I think). And in good news, looks like all relevant OSS-Fuzz reported issues were verified resolved by this PR!!! |
|
Excellent. Yes, I can view the existing issues now, and I'm receiving updates via email. I'll keep working to resolve the Ion-related issues. |
Closes #469
Closes #471
Closes #473
ion-java 1.11.2 release notes: https://github.com/amazon-ion/ion-java/releases/tag/v1.11.2
Maven Central release: https://central.sonatype.com/artifact/com.amazon.ion/ion-java/1.11.2
This should take care of the leaked exceptions identified via fuzzing so far. If we find more we will fix them in subsequent ion-java releases.