Index: lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/CheckCredentialTicketBeforeAdvice.java =================================================================== diff -u -r6dedb2b8e7cc50a87e7e0442c108f4e759b1e832 -rf64be8604a4dd3a9b83ed969043d21e36f28a42f --- lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/CheckCredentialTicketBeforeAdvice.java (.../CheckCredentialTicketBeforeAdvice.java) (revision 6dedb2b8e7cc50a87e7e0442c108f4e759b1e832) +++ lams_contentrepository/src/java/org/lamsfoundation/lams/contentrepository/CheckCredentialTicketBeforeAdvice.java (.../CheckCredentialTicketBeforeAdvice.java) (revision f64be8604a4dd3a9b83ed969043d21e36f28a42f) @@ -49,22 +49,32 @@ 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];