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

multipart cleanups jetty 9 #9345

Merged
merged 5 commits into from
Feb 14, 2023
Merged

Conversation

lachlan-roberts
Copy link
Contributor

bring back some of the improvements to multipart from #9287

Signed-off-by: Lachlan Roberts <[email protected]>
sbordet
sbordet previously approved these changes Feb 13, 2023
Copy link
Contributor

@sbordet sbordet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two small fixes to exception messages, LGTM.

lachlan-roberts and others added 3 commits February 14, 2023 09:34
Co-authored-by: Simone Bordet <[email protected]>
Co-authored-by: Simone Bordet <[email protected]>
Signed-off-by: Lachlan Roberts <[email protected]>
@lachlan-roberts lachlan-roberts merged commit c302159 into jetty-9.4.x Feb 14, 2023
@lachlan-roberts lachlan-roberts deleted the jetty-9.4.x-multipartCleanup branch February 14, 2023 01:15
srowen pushed a commit to apache/spark that referenced this pull request Mar 1, 2023
### What changes were proposed in this pull request?
This pr aims upgrade jetty from 9.4.50.v20221201 to 9.4.51.v20230217.

### Why are the changes needed?
The main change of this version includes:

- jetty/jetty.project#9352
- jetty/jetty.project#9345

The release notes as follows:

- https://github.com/eclipse/jetty.project/releases/tag/jetty-9.4.51.v20230217

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
Pass Github Actions

Closes #40214 from LuciferYang/jetty-9451.

Authored-by: yangjie01 <[email protected]>
Signed-off-by: Sean Owen <[email protected]>
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.

3 participants