Sure. Updated the pull.



-Misagh





From: LELEU Jérôme [mailto:notifications@github.com]
Sent: Sunday, March 17, 2013 1:46 AM
To: Jasig/cas
Cc: Misagh Moayyed
Subject: Re: [cas] CAS-1265: Improve log messages of CASImpl (#210)



In 
cas-server-core/src/main/java/org/jasig/cas/CentralAuthenticationServiceImpl.java:

> @@ -217,7 +216,9 @@ public String grantServiceTicket(final String 
> ticketGrantingTicketId, final Serv
>          final List<Authentication> authns = 
> ticketGrantingTicket.getChainedAuthentications();
>          if(authns.size() > 1) {
>              if (!registeredService.isAllowedToProxy()) {
> -                final String message = String.format("ServiceManagement: 
> Service Attempted to Proxy, but is not allowed. Service: [%s] | Registered 
> Service: [%s]", service.getId(), registeredService.toString());
> +                final String message = String.format("ServiceManagement: 
> Service [%s] (with registered service " +

I understand, but could the line be split another way like this :
final String message = String.
format("ServiceManagement: Service Attempted to Proxy, but is not allowed. 
Service: [%s] | Registered Service: [%s]",
service.getId(), registeredService.toString()); ?

—
Reply to this email directly or view it on GitHub 
<https://github.com/Jasig/cas/pull/210/files#r3405071> . 
<https://github.com/notifications/beacon/xIh4VQSLXYRoprDgzIcjM3ViHa8ww2gENdPt93LRz1dvRkjeaqEc5IAImMv_W-rE.gif>