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

Fix testsuite to work correctly with interoperability handshake #110

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

rachmatowicz
Copy link
Collaborator

@rachmatowicz rachmatowicz commented Apr 6, 2023

This PR does the following:

  • fixes a typo in the system property used to enable EE interoperability mode in the testsuite
  • defines a new method in the HttpTestServer to allow registering a singleton PathHandler to be wrapped (and so participate in the interoperability handshake) and registered as a service handler (c.f. /wildfly-services) in Undertow
  • fixes an issue with handling UNAUTHORIZED responses to the first request across a connection which can cause the interoperability handshake to fail

For more details, see: https://issues.redhat.com/browse/WEJBHTTP-110

@rachmatowicz rachmatowicz force-pushed the WEJBHTTP-110 branch 2 times, most recently from e36b56a to 1b4eda4 Compare June 28, 2023 12:26
@fl4via
Copy link
Collaborator

fl4via commented Jun 30, 2023

@rachmatowicz I assume this is not ready for merge yet, given the comments you raised in the Jira, am I correct?
Still, I just want to let you know that, so far, it looks good to me. Thanks for the hardwork!

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