You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@knox.apache.org by lm...@apache.org on 2017/08/12 15:45:29 UTC

knox git commit: KNOX-995 - KnoxSSO Form IDP Issue with Firefox

Repository: knox
Updated Branches:
  refs/heads/v0.13.0 8613c9689 -> 06f5994a3


KNOX-995 - KnoxSSO Form IDP Issue with Firefox

Project: http://git-wip-us.apache.org/repos/asf/knox/repo
Commit: http://git-wip-us.apache.org/repos/asf/knox/commit/06f5994a
Tree: http://git-wip-us.apache.org/repos/asf/knox/tree/06f5994a
Diff: http://git-wip-us.apache.org/repos/asf/knox/diff/06f5994a

Branch: refs/heads/v0.13.0
Commit: 06f5994a34c80d06e7947506aeefdc7fa3bb61d2
Parents: 8613c96
Author: Larry McCay <lm...@hortonworks.com>
Authored: Sat Aug 12 11:45:18 2017 -0400
Committer: Larry McCay <lm...@hortonworks.com>
Committed: Sat Aug 12 11:45:18 2017 -0400

----------------------------------------------------------------------
 .../resources/applications/knoxauth/app/js/knoxauth.js   | 11 +----------
 gateway-release/src/assembly.xml                         |  3 ++-
 2 files changed, 3 insertions(+), 11 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/knox/blob/06f5994a/gateway-applications/src/main/resources/applications/knoxauth/app/js/knoxauth.js
----------------------------------------------------------------------
diff --git a/gateway-applications/src/main/resources/applications/knoxauth/app/js/knoxauth.js b/gateway-applications/src/main/resources/applications/knoxauth/app/js/knoxauth.js
index 13c2051..57bfff2 100644
--- a/gateway-applications/src/main/resources/applications/knoxauth/app/js/knoxauth.js
+++ b/gateway-applications/src/main/resources/applications/knoxauth/app/js/knoxauth.js
@@ -18,7 +18,6 @@
 var loginPageSuffix = "/knoxauth/login.html";
 var webssoURL = "/api/v1/websso?originalUrl=";
 var userAgent = navigator.userAgent.toLowerCase();
-var firstLogIn = true;
 
 function get(name){
     //KNOX-820 changing the regex so that multiple query params get included with the 'originalUrl'
@@ -84,13 +83,5 @@ var login = function() {
         }
     }
 
-    if (userAgent.indexOf("firefox") != -1){ //TODO: check version number
-        if (firstLogIn) _login();
-        else logoff(_login);
-    }
-    else{
-        _login();
-    }
-
-    if (firstLogIn) firstLogIn = false;
+    _login();
 }
\ No newline at end of file

http://git-wip-us.apache.org/repos/asf/knox/blob/06f5994a/gateway-release/src/assembly.xml
----------------------------------------------------------------------
diff --git a/gateway-release/src/assembly.xml b/gateway-release/src/assembly.xml
index ebbc093..83a5428 100644
--- a/gateway-release/src/assembly.xml
+++ b/gateway-release/src/assembly.xml
@@ -106,6 +106,7 @@
                 <exclude>${gateway-group}:gateway-server-launcher</exclude>
                 <exclude>${gateway-group}:gateway-shell-launcher</exclude>
                 <exclude>${gateway-group}:gateway-demo-ldap-launcher</exclude>
+                <exclude>${gateway-group}:gateway-applications</exclude>
             </excludes>
         </dependencySet>
         <dependencySet>
@@ -144,4 +145,4 @@
             </includes>
         </dependencySet>
     </dependencySets>
-</assembly>
\ No newline at end of file
+</assembly>