-
Notifications
You must be signed in to change notification settings - Fork 251
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
core: upgrade to Jetty 11.0.8 #895
Conversation
@jbonofre can you sign the CLA? |
Hey @jimmarino. Actually I already signed CLA/ECA while ago, just updated it this morning. Let me double check and trigger a new check CI to verify it's OK now. Thanks ! |
@jbonofre for some reason it didn't come through yet |
OK, almost there. Can you please add an entry in the Milestone 3 section of the changelog? |
1 similar comment
OK, almost there. Can you please add an entry in the Milestone 3 section of the changelog? |
@jimmarino sure, let me do that. Thanks ! |
And also, I gonna check about ECA/ICLA again. |
Hi @jbonofre , |
Hey. Sorry for the delay. I will rebase and update the PR. My CLA should be sorted out now. I will rework on it today/tomorrow. I also have other changes in progress (for new PRs). |
1 similar comment
Hey. Sorry for the delay. I will rebase and update the PR. My CLA should be sorted out now. I will rework on it today/tomorrow. I also have other changes in progress (for new PRs). |
Rebased. |
OK, I think I got it: my commit email address is not the same as in my ECA. Trying to rebase/squash with the correct email address. |
ECA OK now :) |
@bscholtes1A @jimmarino can you guys take a look on this one ? Now all is OK, I will move forward with other PRs. |
The dependency analysis looked to have failed due to Java heap space (it works on my machine). |
@bscholtes1A thanks ! I'm moving on other PRs and issues. |
What this PR changes/adds
Upgrade to jetty 11.0.8
Why it does that
Jetty 11.0.8 includes couple of CVE and several bug fixes.
Issue
This closes #896