Re:cvs commit: jakarta-tomcat-connectors/jk/java/org/apache/

2002-02-15 Thread Remy Maucherat
> It looks like you accidently removed the empty string check needed here for > ajp.RemoteUser (). > ajp.RemoteUser () should probably also be checked whether it is null before > calling toString. No, I did it on purpose. It's a lot better to have the user set the behavior of the connector here.

Re:cvs commit: jakarta-tomcat-connectors/jk/java/org/apache/

2002-02-15 Thread Jonathan Pierce
It looks like you accidently removed the empty string check needed here for ajp.RemoteUser (). ajp.RemoteUser () should probably also be checked whether it is null before calling toString. //if ((!(((Ajp13Connector) connector).getTomcatAuthentication())) //&& (ajp.remoteUser() != null)) { //