Index: lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/CheckCredentialTicketBeforeAdvice.java =================================================================== diff -u -rf64be8604a4dd3a9b83ed969043d21e36f28a42f -r6eae08f32a90e2c0b25a2fb5ebfe5b4bd4d6eae5 --- lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/CheckCredentialTicketBeforeAdvice.java (.../CheckCredentialTicketBeforeAdvice.java) (revision f64be8604a4dd3a9b83ed969043d21e36f28a42f) +++ lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/CheckCredentialTicketBeforeAdvice.java (.../CheckCredentialTicketBeforeAdvice.java) (revision 6eae08f32a90e2c0b25a2fb5ebfe5b4bd4d6eae5) @@ -49,32 +49,22 @@ throws AccessDeniedException, RepositoryRuntimeException { // assume that the first argument is the ticket or credential - /** - * Fiona, uncommented the logger below, July, 19th, 2005 Ozgur - */ - - /* if ( log.isDebugEnabled() ) { log.debug("Method " +( m!=null ? m.getName() : "null") +" Checking credential/ticket " +( args != null && args.length > 0 ? args[0] : "null")); } - */ + + if ( m!=null && "toString".equals(m.getName()) ) { + // don't check toString - let it through + return; + } - /** - * Fiona, Should access to repository be denied when this occurs? , July, 19th, 2005 Ozgur - */ if ( args == null || args[0] == null ) { - - log.debug("No ticket/credential supplied. Should access to repository be denied: args:" + args); - if (args != null) - { - log.debug("No ticket/credential supplied. Should access to repository be denied: args[0]:" + args[0]); - } - /** no more throw exception */ - //throw new AccessDeniedException("No ticket/credential supplied. Access to repository denied."); + throw new AccessDeniedException("No ticket/credential supplied. Access to repository denied."); + } else { Object obj = args[0];