https://issues.apache.org/bugzilla/show_bug.cgi?id=45392





--- Comment #5 from Aristotelis <[EMAIL PROTECTED]>  2008-07-14 05:18:15 PST ---
(In reply to comment #2)
> Few things, about the patch.
> 1. Please use 4 spaces instead TAB
> 2. New files must have ASL 2.0 license header at the top.
> 
> Please make the patch against the svn trunk.

  I'll fix these issues, create a new patch and upload it here. 

> Some documentation would be great if additional setup is required.
> 

  Actually at this time since the patch uses the ocsp server info found in the
certificates no further setup is required. If extra functionality is to be
introduced (i.e having a default ocsp server) perhaps some additional setup
must be made.

> 
> Other then that, the patch is pretty promissing.
> It would be nice to use the APR for make_socket, but that can be done later.
> 

  I have to find some documentation and/or look on some code on how to use APR
for the socket.

 Thanks :)


-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

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

Reply via email to