details:   https://code.openbravo.com/erp/devel/pi/rev/4f9959e58bd3
changeset: 29025:4f9959e58bd3
user:      Asier Lostalé <asier.lostale <at> openbravo.com>
date:      Thu Apr 14 10:16:14 2016 +0200
summary:   related to bug 32663: instabilities in int-gui job

  Added extra log when trying to log with incorrect client/org

diffstat:

 src/org/openbravo/base/secureApp/LoginUtils.java |  16 +++++++++-------
 1 files changed, 9 insertions(+), 7 deletions(-)

diffs (40 lines):

diff -r 7241af045c9e -r 4f9959e58bd3 
src/org/openbravo/base/secureApp/LoginUtils.java
--- a/src/org/openbravo/base/secureApp/LoginUtils.java  Wed Apr 13 16:20:51 
2016 +0200
+++ b/src/org/openbravo/base/secureApp/LoginUtils.java  Thu Apr 14 10:16:14 
2016 +0200
@@ -1,6 +1,6 @@
 /*
  
************************************************************************************
- * Copyright (C) 2001-2014 Openbravo S.L.U.
+ * Copyright (C) 2001-2016 Openbravo S.L.U.
  * Licensed under the Apache Software License version 2.0
  * You may obtain a copy of the License at 
http://www.apache.org/licenses/LICENSE-2.0
  * Unless required by applicable law or agreed to  in writing,  software  
distributed
@@ -120,20 +120,22 @@
     return valid;
   }
 
-  static boolean validRoleClient(ConnectionProvider conn, String strRol, 
String strCliente)
+  static boolean validRoleClient(ConnectionProvider conn, String role, String 
client)
       throws ServletException {
-    boolean valid = SeguridadData.isRoleClient(conn, strRol, strCliente);
+    boolean valid = SeguridadData.isRoleClient(conn, role, client);
     if (!valid) {
-      log4j.error("Login client is not in role clients list");
+      log4j
+          .error("Login client is not in role clients list. Role: " + role + 
", Client: " + client);
     }
     return valid;
   }
 
-  static boolean validRoleOrg(ConnectionProvider conn, String strRol, String 
strOrg)
+  static boolean validRoleOrg(ConnectionProvider conn, String role, String org)
       throws ServletException {
-    boolean valid = SeguridadData.isLoginRoleOrg(conn, strRol, strOrg);
+    boolean valid = SeguridadData.isLoginRoleOrg(conn, role, org);
     if (!valid) {
-      log4j.error("Login organization is not in role organizations list");
+      log4j.error("Login organization is not in role organizations list. Role: 
" + role + ", Org: "
+          + org);
     }
     return valid;
   }

------------------------------------------------------------------------------
Find and fix application performance issues faster with Applications Manager
Applications Manager provides deep performance insights into multiple tiers of
your business applications. It resolves application problems quickly and
reduces your MTTR. Get your free trial!
https://ad.doubleclick.net/ddm/clk/302982198;130105516;z
_______________________________________________
Openbravo-commits mailing list
Openbravo-commits@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openbravo-commits

Reply via email to