Skip to content

Implement Jetty ComplianceViolation.Listener - #532

Open
olamy wants to merge 4 commits into
jenkinsci:masterfrom
olamy:compliance-violation-listener
Open

Implement Jetty ComplianceViolation.Listener#532
olamy wants to merge 4 commits into
jenkinsci:masterfrom
olamy:compliance-violation-listener

Conversation

@olamy

@olamy olamy commented Mar 24, 2026

Copy link
Copy Markdown
Member

Signed-off-by: Olivier Lamy olamy@apache.org

Testing done

Submitter checklist

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests that demonstrate the feature works or the issue is fixed

@jglick jglick left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks useful! Untested. Does not help with the HTTP/2 upgrade noted in jetty/jetty.project#14730.

Comment thread src/main/java/winstone/WinstoneViolationListener.java Outdated
Comment thread src/main/java/winstone/WinstoneViolationListener.java Outdated
Comment thread src/test/java/winstone/WinstoneViolationListenerTest.java Outdated
olamy added 3 commits March 25, 2026 14:16
Signed-off-by: Olivier Lamy <olamy@apache.org>
Signed-off-by: Olivier Lamy <olamy@apache.org>
Signed-off-by: Olivier Lamy <olamy@apache.org>
@olamy
olamy force-pushed the compliance-violation-listener branch from d09f597 to 8541281 Compare March 25, 2026 04:28
Signed-off-by: Olivier Lamy <olamy@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants