Skip to content
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

[GHSA-qw69-rqj8-6qw8] OutOfMemoryError for large multipart without filename in Eclipse Jetty #2393

Conversation

jeffalder
Copy link

Updates

  • Affected products

Comments
This GHSA is showing as a false positive when we have the documented, patched versions specified in our dependency list.

@github
Copy link
Collaborator

github commented Jun 9, 2023

Hi there @sbordet and @jmcc0nn3ll! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.

This change will be reviewed by our highly-trained Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory

@github-actions github-actions bot changed the base branch from main to jeffalder/advisory-improvement-2393 June 9, 2023 18:18
@jeffalder
Copy link
Author

https://jubianchi.github.io/semver-check/#/%3C9.4.51.v20230217/9.4.51

If semver is being used, then GH's semver needs to adapt to handle more-generalized version strings.

@jeffalder
Copy link
Author

https://central.sonatype.com/artifact/org.eclipse.jetty/jetty-server/9.4.51.v20230217 - Jetty Server, for the 9.4 series, uses this date-based numbering for all its versions, so I'll never see a 9.4.51 or 9.4.52.

@darakian
Copy link
Contributor

Hey @jeffalder, sorry about that and thanks for the update. In general patch notes/fix commits/etc... help us process these faster.
eg. https://github.com/eclipse/jetty.project/releases/tag/jetty-9.4.51.v20230217

@advisory-database advisory-database bot merged commit 668bc1c into jeffalder/advisory-improvement-2393 Jun 12, 2023
2 checks passed
@advisory-database
Copy link
Contributor

Hi @jeffalder! Thank you so much for contributing to the GitHub Advisory Database. This database is free, open, and accessible to all, and it's people like you who make it great. Thanks for choosing to help others. We hope you send in more contributions in the future!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants