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 #6883 relative welcome redirect #6886
Fix #6883 relative welcome redirect #6886
Conversation
Fix #6883 relative welcome redirect + make all redirects able to be relative + added test for relative redirection in ResourceService Signed-off-by: Greg Wilkins <gregw@webtide.com>
@gregw the new tests do not seem to test the original issue. I locally reverted the changes to |
@lachlan-roberts good catch. There is a bug in |
Fix the HttpFieldsContainsHeaderValue impl Signed-off-by: Greg Wilkins <gregw@webtide.com>
@lachlan-roberts I've pushed a fix for that, but there is a good chance it might break some other tests that were relying on the wrong behavior |
jetty-http/src/test/java/org/eclipse/jetty/http/matchers/HttpFieldsContainsHeaderValue.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Greg Wilkins <gregw@webtide.com>
…83-relative-welcome-redirect
Signed-off-by: Greg Wilkins <gregw@webtide.com>
Fix #6883 relative welcome redirect
Signed-off-by: Greg Wilkins gregw@webtide.com