Skip to content

Commit 73dc342

Browse files
committed
Bugfix - Getting target url from UrlProvider
1 parent d0c0bc0 commit 73dc342

File tree

2 files changed

+11
-14
lines changed

2 files changed

+11
-14
lines changed

QueueIT.Security/src/queueit/security/KnownUserFactory.java

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -65,6 +65,15 @@ public static void configure(
6565
defaultQuerystringPrefix = querystringPrefix;
6666
}
6767
}
68+
69+
static IKnownUserUrlProvider getKnownUserUrlProvider()
70+
{
71+
try {
72+
return defaultUrlProviderFactory.call();
73+
} catch (Exception ex) {
74+
throw new InvalidKnownUserUrlException();
75+
}
76+
}
6877

6978
public static IKnownUser verifyMd5Hash()
7079
throws InvalidKnownUserUrlException, InvalidKnownUserHashException {

QueueIT.Security/src/queueit/security/Queue.java

Lines changed: 2 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -323,25 +323,13 @@ private UriComponentsBuilder getQueueUrlWithoutTarget(Boolean sslEnabled, String
323323

324324
private void includeTargetUrl(Boolean includeTargetUrl, UriComponentsBuilder queueUrl)
325325
{
326-
HttpServletRequest request = RequestContext.getCurrentInstance().getRequest();
327-
if (request == null) {
328-
return;
329-
}
330-
331326
if (includeTargetUrl == null)
332327
includeTargetUrl = this.defaultIncludeTargetUrl;
333328

334329
if (!includeTargetUrl)
335330
return;
336-
337-
StringBuffer requestURL = request.getRequestURL();
338-
String queryString = request.getQueryString();
339-
340-
if (queryString != null) {
341-
requestURL.append('?').append(queryString);
342-
}
343-
344-
includeTargetUrl(requestURL.toString().toString(), queueUrl);
331+
332+
includeTargetUrl(KnownUserFactory.getKnownUserUrlProvider().getUrl(), queueUrl);
345333
}
346334

347335
private static void includeTargetUrl(String targetUrl, UriComponentsBuilder queueUrl)

0 commit comments

Comments
 (0)