Skip to content

Commit

Permalink
wevdav: fix redirect path
Browse files Browse the repository at this point in the history
Motivation:

Commit 8a4273f seem to have introduced
regression in how redirect paths are presented.

Modification:

Pass request.getRequestURL() to call to ServletRequest.stripToPath
instead of request.getRequestURI()

Result:

Observe correct path on redirect

Patch: https://rb.dcache.org/r/14130/
Target: trunk
Request: 9.2
Request: 9.1
Request: 9.0
Request: 8.2
  • Loading branch information
DmitryLitvintsev committed Oct 10, 2023
1 parent ce9734a commit c65c007
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -1693,7 +1693,7 @@ public HttpTransfer(PnfsHandler pnfs, Subject subject,

var request = ServletRequest.getRequest();
request.setAttribute(TRANSACTION_ATTRIBUTE, getTransaction());
_requestPath = ServletRequest.stripToPath(request.getRequestURI());
_requestPath = ServletRequest.stripToPath(request.getRequestURL().toString());
}

protected ProtocolInfo createProtocolInfo(InetSocketAddress address) {
Expand Down

0 comments on commit c65c007

Please sign in to comment.