Project:
| View Revisions: Issue #36582 | [ All Revisions ] [ Back to Issue ] | ||
| Summary | 0036582: Bad logging code at exception at MobileCoreLoginHandler | ||
| Revision | 2017-08-03 10:23 by ioritzCia | ||
| Description | There is bad logging code at MobileCoreLoginHandler. In https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/file/e3cb74a79b0d/src/org/openbravo/mobile/core/login/MobileCoreLoginHandler.java [^] Line 198 should disappear, it is wrong and in production systems it is printing errors in catalina.out Line 199 should be a debug and should not print the stack trace when the exception is because of wrong password. Line 204 should be reviewed as well when the exception is because of wrong password. For lines 199 and 204 the behaviour should only change when it is an identification error, not for the rest of the error. There are two files uploaded with proposed changes to take out the printStackTraces from posterminal and mobilecore. |
||
| Revision | 2017-08-02 17:22 by ioritzCia | ||
| Description | There is bad logging code at MobileCoreLoginHandler. In https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/file/e3cb74a79b0d/src/org/openbravo/mobile/core/login/MobileCoreLoginHandler.java [^] Line 198 should disappear, it is wrong and in production systems it is printing errors in catalina.out Line 199 should be a debug and should not print the stack trace when the exception is because of wrong password. Line 204 should be reviewed as well when the exception is because of wrong password. There are two files uploaded with proposed changes to take out the printStackTraces from posterminal and mobilecore. |
||
| Revision | 2017-08-02 17:02 by ioritzCia | ||
| Description | There is bad logging code at MobileCoreLoginHandler. In https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/file/e3cb74a79b0d/src/org/openbravo/mobile/core/login/MobileCoreLoginHandler.java [^] Line 198 should disappear, it is wrong and in production systems it is printing errors in catalina.out Line 199 should be a debug and should not print the stack trace when the exception is because of wrong password. Line 204 should be reviewed as well when the exception is because of wrong password. |
||
| Revision | 2017-08-02 16:55 by ioritzCia | ||
| Description | There is bad logging code at MobileCoreLoginHandler. In https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/file/e3cb74a79b0d/src/org/openbravo/mobile/core/login/MobileCoreLoginHandler.java [^] Line 198 should disappear, it is wrong and in production systems it is printing errors in catalina.out Line 199 should be a debug and should not print the stack trace. Line 204 should be reviewed as well. |
||
| Revision | 2017-08-02 15:41 by ioritzCia | ||
| Description | There is bad logging code at MobileCoreLoginHandler. In https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/file/e3cb74a79b0d/src/org/openbravo/mobile/core/login/MobileCoreLoginHandler.java [^] Line 198 should disappear, it is wrong and in production systems it is printing errors in catalina.out Line 199 should be a debug and should not print the stack trace. |
||
| Revision | 2017-08-02 15:41 by ioritzCia | ||
| Description | There is bad logging code at MobileCoreLoginHandler. In https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/file/e3cb74a79b0d/src/org/openbravo/mobile/core/login/MobileCoreLoginHandler.java [^] Line 198 should disappear it is wrong and in production systems it is printing errors in catalina.out Line 199 should be a debug and should not print the stack trace. |
||
| Copyright © 2000 - 2009 MantisBT Group |
 |