This is an automated email from the ASF dual-hosted git repository.
smolnar pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/knox.git
The following commit(s) were added to refs/heads/master by this push:
new d56937358 KNOX-2979 - Removed redundant 'refresh' query parameter from
the application logout link after originalUrl (#815)
d56937358 is described below
commit d569373582b800c6c0346cb25e5fe5ec06054050
Author: Sandor Molnar <[email protected]>
AuthorDate: Tue Oct 31 15:12:59 2023 +0100
KNOX-2979 - Removed redundant 'refresh' query parameter from the
application logout link after originalUrl (#815)
---
.../main/resources/applications/knoxauth/app/logout.jsp | 8 ++------
knox-homepage-ui/home/app/homepage.service.ts | 14 --------------
2 files changed, 2 insertions(+), 20 deletions(-)
diff --git
a/gateway-applications/src/main/resources/applications/knoxauth/app/logout.jsp
b/gateway-applications/src/main/resources/applications/knoxauth/app/logout.jsp
index 27ddc9602..51363be3b 100644
---
a/gateway-applications/src/main/resources/applications/knoxauth/app/logout.jsp
+++
b/gateway-applications/src/main/resources/applications/knoxauth/app/logout.jsp
@@ -90,17 +90,13 @@
boolean validRedirect = false;
String origUrl = request.getParameter("originalUrl");
- String del = "?";
- if (origUrl != null && origUrl.contains("?")) {
- del = "&";
- }
if (origUrl != null) {
validRedirect = RegExUtils.checkWhitelist(whitelist, origUrl);
}
if (("1".equals(request.getParameter("returnToApp")))) {
if (validRedirect) {
- response.setStatus(HttpServletResponse.SC_MOVED_PERMANENTLY);
- response.setHeader("Location",originalUrl + del + "refresh=1");
+ response.setStatus(HttpServletResponse.SC_MOVED_PERMANENTLY);
+ response.setHeader("Location", originalUrl);
return;
}
}
diff --git a/knox-homepage-ui/home/app/homepage.service.ts
b/knox-homepage-ui/home/app/homepage.service.ts
index 123e762d3..cbad3d580 100644
--- a/knox-homepage-ui/home/app/homepage.service.ts
+++ b/knox-homepage-ui/home/app/homepage.service.ts
@@ -132,20 +132,6 @@ export class HomepageService {
}
private handleError(error: HttpErrorResponse): Promise<any> {
- // location.reload();
- let refresh;
- this.route.queryParams.subscribe(params => {
- refresh = params['refresh'];
- console.debug('refresh = ' + refresh);
- if (refresh) {
- console.debug('Refreshing page...', window.location.href);
- let url = window.location.pathname.replace(new
RegExp('refresh=1/.*'), '?');
- // var url = window.location.pathname;
-
- // window.location.assign(url);
- window.location.reload();
- }
- });
Swal.fire({
icon: 'error',
title: 'Oops!',
