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

Fixing servlet ResponseWriter#writeResponseStatusAndHeaders for error states #5435

Merged
merged 1 commit into from
Oct 17, 2023

Conversation

jansupol
Copy link
Contributor

Bug #5279

… states

Bug 5279

Signed-off-by: jansupol <jan.supol@oracle.com>
@senivam senivam merged commit a837087 into eclipse-ee4j:2.x Oct 17, 2023
2 of 3 checks passed
@senivam senivam added this to the 2.41 milestone Oct 17, 2023
@jansupol jansupol deleted the 5415 branch October 18, 2023 13:19
This was referenced Oct 20, 2023
srowen pushed a commit to apache/spark that referenced this pull request Oct 29, 2023
### What changes were proposed in this pull request?
This pr aims upgrade Jersey from 2.40 to 2.41.

### Why are the changes needed?
The new version bring some improvements, like:
- eclipse-ee4j/jersey#5350
- eclipse-ee4j/jersey#5365
- eclipse-ee4j/jersey#5436
- eclipse-ee4j/jersey#5296

and some bug fix, like:
- eclipse-ee4j/jersey#5359
- eclipse-ee4j/jersey#5405
- eclipse-ee4j/jersey#5423
- eclipse-ee4j/jersey#5435
- eclipse-ee4j/jersey#5445

The full release notes as follows:
- https://github.com/eclipse-ee4j/jersey/releases/tag/2.41

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

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

### Was this patch authored or co-authored using generative AI tooling?
No

Closes #43490 from LuciferYang/SPARK-45636.

Lead-authored-by: YangJie <yangjie01@baidu.com>
Co-authored-by: yangjie01 <yangjie01@baidu.com>
Signed-off-by: Sean Owen <srowen@gmail.com>
@peter-janssen
Copy link

Not sure what the process is here but shouldn't this be forward ported to 3.0.x and 3.1.x?

@jansupol
Copy link
Contributor Author

@peter-janssen Do not worry, it is done. Feel free to check sources.

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