attila-kun opened a new pull request #255:
URL: https://github.com/apache/qpid-proton/pull/255


   Steps to reproduce:
   1. Connect your receiver to an SSL-capable broker. The receiver has to be 
running on Windows.
   2. Set the `idle_timeout` connection option to e.g. 2 seconds in your 
`proton::messaging_handler` instance.
   2. In the `on_message` callback, sleep for longer than `idle_timeout`. This 
should trigger the `on_transport_error` callback with a 
`amqp:resource-limit-exceeded: local-idle-timeout expired` error. On Windows 
this does not happen.
   
   I saw that the above scenario worked correctly when connecting to a non-SSL 
broker but failed with an SSL broker. I looked at the difference between the 
two code paths and found that for the same input (`transport->output_buffer` 
being empty and `transport->close_sent == true`) [`transport.c` returns 
`PN_EOS`](https://github.com/apache/qpid-proton/blob/0.31.0/c/src/core/transport.c#L2722)
 while `schannel.c` returned `0`. After ensuring that `schannel.c` matches the 
behaviour of `transport.c`, the problem went away.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to