Help Desk Ticket Reference: https://c.na7.visual.force.com/apex/Case_View?id=500A0000008Qeyd&sfdc.override=1 project_key: JBEPP org.picketlink.idm.impl.repository.FallbackIdentityStoreRepository doesn't throw an ERROR exception when a ldap connection has timed out. To see that exception user needs to enable the INFO logging to "org.picketlink" category, then it prints the failed connection attempt successfully. 2011-09-19 11:54:44,315 INFO [org.picketlink.idm.impl.repository.FallbackIdentityStoreRepository] (main) Failed to create IdentityObject: org.picketlink.idm.common.exception.IdentityException: Could not obtain LDAP connection: Caused by: javax.naming.CommunicationException: [host]:[port] [Root exception is java.net.SocketTimeoutException: connect timed out] This enhancement is to change the log level of from INFO to ERROR and to improve the log messages in ./picketlink-idm/picketlink-idm-core/src/main/java/org/picketlink/idm/impl/repository/FallbackIdentityStoreRepository.java Instead of using INFO it would be easier to see the issue with an ERROR log level. Here is an example of log to be improved/changed to ERROR: try { [source] } catch (IdentityException e) { if (log.isLoggable(Level.INFO)) { log.log(Level.INFO, "Failed to create IdentityObject: ", e); } }
Release Notes Docs Status: Added: Not Yet Documented
Labels: Added: EPP_5_2_1_Candidate
I've commited fix into Picketlink IDM trunk - https://source.jboss.org/changelog/PicketLink?cs=1367 . I added SEVERE logging instead of INFO for all exceptions, which are not propagated to the caller. For those, which are propagated there is FINER logging, but it seems to be sufficient since the exception should be processed and logged by the caller. btw. I used SEVERE category instead of ERROR, because Picketlink IDM is using JDK logging and it does not support ERROR category (categories are SEVERE, WARN, INFO, FINE, FINER, FINEST)
Only needed action is to release Picketlink IDM and update dependency in EPP pom.xml.
Labels: Removed: EPP_5_2_1_Candidate
Technical note added. If any revisions are required, please edit the "Technical Notes" field accordingly. All revisions will be proofread by the Engineering Content Services team. New Contents: Release notes docs status: Not Yet Documented Release notes text: CAUSE: Class FallbackIdentityStoreRepository from library Picketlink IDM was using insufficient logging in some places, because some errors were reported into log with log level INFO, which may cause that customer can miss some important error messages. FIX: It's fixed in library Picketlink IDM. I added SEVERE logging instead of INFO for all exceptions, which are not propagated to the caller. For those, which are propagated there is FINER logging, but it seems to be sufficient since the exception should be processed and logged by the caller. btw. I used SEVERE category instead of ERROR, because Picketlink IDM is using JDK logging and it does not support ERROR category (categories are SEVERE, WARN, INFO, FINE, FINER, FINEST)
Technical note updated. If any revisions are required, please edit the "Technical Notes" field accordingly. All revisions will be proofread by the Engineering Content Services team. Diffed Contents: @@ -1,6 +1,4 @@ -Release notes docs status: Not Yet Documented - -Release notes text: CAUSE: Class FallbackIdentityStoreRepository from library Picketlink IDM was using insufficient logging in some places, because some errors were reported into log with log level INFO, which may cause that customer can miss some important error messages. +CAUSE: Class FallbackIdentityStoreRepository from library Picketlink IDM was using insufficient logging in some places, because some errors were reported into log with log level INFO, which may cause that customer can miss some important error messages. FIX: It's fixed in library Picketlink IDM. I added SEVERE logging instead of INFO for all exceptions, which are not propagated to the caller. For
Technical note updated. If any revisions are required, please edit the "Technical Notes" field accordingly. All revisions will be proofread by the Engineering Content Services team. Diffed Contents: @@ -1,10 +1 @@ -CAUSE: Class FallbackIdentityStoreRepository from library Picketlink IDM was using insufficient logging in some places, because some errors were reported into log with log level INFO, which may cause that customer can miss some important error messages. +It was discovered that the Picketlink IDM FallbackIdentityStoreRepository class was using INFO logging instead of an elevated log level. Customers could miss important error information in the logs because INFO is commonly used for non-critical errors. The fix implements SEVERE logging category changes for all exceptions that are not propagated to the caller. For exceptions that are propagated to the caller, FINER logging is configured because these exceptions should be processed and logged by the caller.- -FIX: It's fixed in library Picketlink IDM. I added SEVERE logging -instead of INFO for all exceptions, which are not propagated to the caller. For -those, which are propagated there is FINER logging, but it seems to be -sufficient since the exception should be processed and logged by the caller. - -btw. I used SEVERE category instead of ERROR, because Picketlink IDM is using -JDK logging and it does not support ERROR category (categories are SEVERE, -WARN, INFO, FINE, FINER, FINEST)