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

Moved implementation methods ensure*() from the Response interface #9390

Merged
merged 1 commit into from
Feb 16, 2023

Conversation

sbordet
Copy link
Contributor

@sbordet sbordet commented Feb 16, 2023

to ResponseUtils to avoid cluttering the main Response interface.

to ResponseUtils to avoid cluttering the main Response interface.

Signed-off-by: Simone Bordet <[email protected]>
@sbordet sbordet requested review from gregw and lorban February 16, 2023 17:52
@gregw gregw merged commit d21f387 into jetty-12.0.x Feb 16, 2023
gregpoulos pushed a commit to gregpoulos/jetty.project that referenced this pull request Feb 18, 2023
… into jetty-12.0.x-old-docs-remove-logging-sections

* 'jetty-12.0.x' of https://github.com/eclipse/jetty.project:
  Issue jetty#9182 - make JakartaWSSCI.initialize() and JakartaWebSocketServerContainer public
  ensure the WebSocketConnection is set on the WebSocketCoreSession
  remove osgi internal imports for websocket-core
  rename WebSocketUtil to WebSocketUtils
  make WebSocketCoreSession public & other fixes
  Javadocs for Response and Context. (jetty#9388)
  Moved implementation methods ensure*() from the Response interface (jetty#9390)
  Reinstate ee9 jetty runner. (jetty#9383)
  Fix jetty#9387
  remove setClassLoader from CoreSession interface
  remove exporting of internal packages in ee9 & ee10 websocket
  fix remaining JPMS issues in websocket-core
  resolve JPMS issues with CoreSession and WebSocketCoreSession
  move websocket-core-common messages and util packages out of internal
@joakime joakime deleted the fix/jetty-12-remove-response-ensure-methods branch March 29, 2023 14:30
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.

2 participants