Remy Maucherat wrote:
> [EMAIL PROTECTED] wrote:
> 
>> bobh        2002/09/18 09:44:35
>>
>>   Modified:    .        gump.xml
>>                coyote/src/java/org/apache/coyote Request.java
>>                http11/src/java/org/apache/coyote/http11
>>                         Http11Processor.java
>>   Log:
>>   - This associates the socket with the Request.  This is so the
>>   CertificatesValve.verify() can tell where (which socket) the Request
>>   is comeing from.  Without this association, CertificatesValve.verify()
>>   returns with no SSL Handshake.
>>   - this change is in part based on feed back from; Vivek N. Yingxian
>>   Wang (JSSE), Craig M., Qingqing Ouyang
> 
> 
> -1 (ie, revert this).
> 
> The socket should already have been associated with the request, but as 
> a note. It is available, and accessible.

Ok, I checked it, and the socket os not currently associated with the 
request (I thought it was). OTOH, you shouldn't add a field for socket 
in the request (native webservers don't have the notion of a Java 
socket). Instead, use one of the notes.

Remy


--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to