> should the if () test be:
> 
> if ((session != null) && session.isValid()) {
>           return session;
> }

Yep, looks like a bug to me...I just checked in this change to the code...

cvs diff
 JServConnection.java
Index: JServConnection.java
===================================================================
RCS file:
/products/cvs/master/jserv/src/java/org/apache/jserv/JServConnection.java,v
retrieving revision 1.45
diff -r1.45 JServConnection.java
1249c1249
<         if (session != null) {
---
>         if (session != null && session.isValid()) {
1255c1255
<           if (session != null) {
---
>           if (session != null && session.isValid()) {
1785c1785
< }---
> }

thanks,

-jon



-- --------------------------------------------------------------
To subscribe:        [EMAIL PROTECTED]
To unsubscribe:      [EMAIL PROTECTED]
READ THE FAQ!!!!     <http://java.apache.org/faq/>
Archives and Other:  <http://java.apache.org/main/mail.html/>
Problems?:           [EMAIL PROTECTED]

Reply via email to