-
-
Notifications
You must be signed in to change notification settings - Fork 8.8k
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
[JENKINS-64035] - Winstone 5.12: Update Jetty from 9.4.30.v20200611 to 9.4.33.v20201020 #5034
Conversation
…o 9.4.33.v20201020
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
@jenkinsci/core I suggest an expedited merge of this pull request into 2.264. It would help us to collect more community feedback and decide whether the new Jetty release can be [threoretically] backported into 2.263.1 |
We may merge it tomorrow at 9AM UTC if there is no negative feedback. Please see the merge process documentation for more information about the merge process |
I suggest just merging after CI is green... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK, taking the feedback. 🚢 🇮🇹 |
The 9.4.33.v20201020 Jetty version picks up an upstream fix for jetty/jetty.project#5417 which caused JENKINS-63958 in Jenkins 2.261 and was worked around by downgrading Jetty in Winstone 5.11.1 and Jenkins 2.263. It would be great to include the new version into the weekly release, and then, maybe, consider it for backporting to 2.263.x LTS if the community ratings are fine.
Proposed changelog entries
Proposed upgrade guidelines
N/A
Submitter checklist
Proposed changelog entries
section only if there are breaking changes or other changes which may require extra steps from users during the upgradeDesired reviewers
@mention
Maintainer checklist
Before the changes are marked as
ready-for-merge
:Proposed changelog entries
are correctupgrade-guide-needed
label is set and there is aProposed upgrade guidelines
section in the PR title. (example)lts-candidate
to be considered (see query).