Bob Herrmann wrote:
> On Wed, 2002-09-18 at 13:58, Remy Maucherat wrote:
> 
>>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.
> 
> 
> Ok, 
>   - change Http11Processor to use a note
>   - change Request.java to it's original form
>   - change the CertificatesValve.verify()
> 
> Does that sound about right?

For me: yes.

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




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

Reply via email to