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

Jump to bottom

getRequestURL can append "null" if getRequestURI is unspecified in an authority-form request-target #8578

Closed
electrum opened this issue Sep 9, 2022 · 3 comments
Closed

getRequestURL can append "null" if getRequestURI is unspecified in an authority-form request-target #8578

electrum opened this issue Sep 9, 2022 · 3 comments
Assignees
@joakime
Labels
Bug For general bugs on Jetty side Sponsored This issue affects a user with a commercial support agreement

Comments

@electrum
Copy link

electrum commented Sep 9, 2022 •
edited

Jetty version(s)

9.4.x
10.x

Description

When upgrading to Jetty 9.4.48, we see that AbstractProxyServlet.rewriteTarget() calls clientRequest.getRequestURL() which results in https://localhost:50393null due to getRequestURI() returning null (since the URI path is null). While this Request behavior seems to have existed for ~10 years and is not the root issue, it still seems to be a bug.
@electrum electrum added the Bug For general bugs on Jetty side label Sep 9, 2022
@joakime joakime added the Sponsored This issue affects a user with a commercial support agreement label Sep 9, 2022
joakime added a commit that referenced this issue Sep 9, 2022
@joakime
Issue #8578 - restore backward compat of getRequestURL and getRequest…
d3c7ee3 
…URI when working with CONNECT method

Signed-off-by: Joakim Erdfelt <joakim.erdfelt@gmail.com>
@joakime joakime linked a pull request Sep 9, 2022 that will close this issue
Issue #8578 - getRequestURL can append "null" if getRequestURI is unspecified in an authority-form request-target #8579
Merged
@joakime
Copy link
Contributor

joakime commented Sep 9, 2022

Opened PR #8579 to solve for jetty-9.4.x.

Attempting to replicate in jetty-10.0.x now ...

joakime added a commit that referenced this issue Sep 9, 2022
@joakime
Issue #8578 - test both request URL/URI results
48c16de 
Signed-off-by: Joakim Erdfelt <joakim.erdfelt@gmail.com>
joakime added a commit that referenced this issue Sep 9, 2022
@joakime
Issue #8578 - restore backward compat of getRequestURL and getRequest…
a6a386a 
…URI when working with CONNECT method

Signed-off-by: Joakim Erdfelt <joakim.erdfelt@gmail.com>
@joakime joakime linked a pull request Sep 9, 2022 that will close this issue
Issue #8578 - getRequestURL can append "null" if getRequestURI is unspecified in an authority-form request-target #8580
Merged
This was unlinked from pull requests Sep 9, 2022
Issue #8578 - getRequestURL can append "null" if getRequestURI is unspecified in an authority-form request-target #8580
Merged
Issue #8578 - getRequestURL can append "null" if getRequestURI is unspecified in an authority-form request-target #8579
Merged
@joakime
Copy link
Contributor

joakime commented Sep 9, 2022

Opened PR #8580 to solve for jetty-10.0.x (this will be merged to jetty-11.0.x once approved/merged)

@joakime joakime self-assigned this Sep 9, 2022
joakime added a commit that referenced this issue Sep 12, 2022
@joakime
Issue #8578 - Changes from review
5944ff4 
Signed-off-by: Joakim Erdfelt <joakim.erdfelt@gmail.com>
joakime added a commit that referenced this issue Sep 12, 2022
@joakime
Issue #8578 - Changes from review
8eba1c9 
Signed-off-by: Joakim Erdfelt <joakim.erdfelt@gmail.com>
joakime added a commit that referenced this issue Sep 13, 2022
@joakime
Merge pull request #8579 from eclipse/fix/jetty-9.4.x-abstractproxy-n…
b32d739 
…ull-request-url

Issue #8578 - `getRequestURL` can append "null" if `getRequestURI` is unspecified in an authority-form request-target
joakime added a commit that referenced this issue Sep 13, 2022
@joakime
Merge pull request #8580 from eclipse/fix/jetty-10.0.x-proxy-rewrite-…
2a49f86 
…null-uri

Issue #8578 - `getRequestURL` can append "null" if `getRequestURI` is unspecified in an authority-form request-target
@joakime
Copy link
Contributor

joakime commented Sep 13, 2022

Complete.

Merged from 9.4.x through to 12.0.x

@joakime joakime closed this as completed Sep 13, 2022
@joakime joakime changed the title Request.getRequestURL() can append null getRequestURI() getRequestURL can append "null" if getRequestURI is unspecified in an authority-form request-target Sep 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@joakime joakime

Labels
Bug For general bugs on Jetty side Sponsored This issue affects a user with a commercial support agreement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@electrum @joakime