You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@knox.apache.org by pz...@apache.org on 2022/02/24 21:38:13 UTC
[knox] branch master updated: KNOX-2706 - Possible NPE in redirecting.jsp (#540)
This is an automated email from the ASF dual-hosted git repository.
pzampino 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 df6dd3a KNOX-2706 - Possible NPE in redirecting.jsp (#540)
df6dd3a is described below
commit df6dd3afc2726b03e66f2c7f74dfb079652364b1
Author: Phil Zampino <pz...@apache.org>
AuthorDate: Thu Feb 24 16:38:06 2022 -0500
KNOX-2706 - Possible NPE in redirecting.jsp (#540)
---
.../src/main/resources/applications/knoxauth/app/redirecting.jsp | 3 +++
1 file changed, 3 insertions(+)
diff --git a/gateway-applications/src/main/resources/applications/knoxauth/app/redirecting.jsp b/gateway-applications/src/main/resources/applications/knoxauth/app/redirecting.jsp
index 4acbd83..2a96117 100644
--- a/gateway-applications/src/main/resources/applications/knoxauth/app/redirecting.jsp
+++ b/gateway-applications/src/main/resources/applications/knoxauth/app/redirecting.jsp
@@ -45,6 +45,9 @@
<%
boolean validRedirect = true;
String originalUrl = request.getParameter("originalUrl");
+ if (originalUrl == null) {
+ originalUrl = "";
+ }
try {
if (Urls.containsUserInfo(originalUrl)) {
validRedirect = false;