NRPE SSL_shutdown patch
Andreas Ericsson
ae at op5.se
Thu Mar 29 14:34:23 CEST 2012
On 03/29/2012 02:14 PM, Jari Takkala wrote:
> On Thu, Mar 29, 2012 at 12:19 PM, Andreas Ericsson<ae at op5.se> wrote:
>> * Write a wrapper to shutdown the connection so you don't have to
>> duplicate the code everywhere. close_connection() would be a good
>> name for it, and it should handle the #ifdef HAVE_SSL stuff as well
>> so it can be removed from (most) of the application logic.
>>
>> * Add a short sleep-timer between calls, or poll() the underlying
>> socket for inbound data so the other node has time to send a
>> response. I imagine your patch will work horribly on links from
>> Europe to China, for instance, because latency will be too high.
>
> Agree with regards to both issues. The code is not in my style, was
> trying to keep it similar to project's.
>
> The attached patch hopefully fixes the duplicate code to your
> satisfaction. If I have some time I'll look at addressing the time-out
> issue later (interestingly, a few other major projects don't take this
> into account either).
>
Possibly nice, except I can't read the attachment, or save it, so I
can't really comment.
--
Andreas Ericsson andreas.ericsson at op5.se
OP5 AB www.op5.se
Tel: +46 8-230225 Fax: +46 8-230231
Considering the successes of the wars on alcohol, poverty, drugs and
terror, I think we should give some serious thought to declaring war
on peace.
------------------------------------------------------------------------------
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here
http://p.sf.net/sfu/sfd2d-msazure
More information about the Developers
mailing list