[ https://issues.apache.org/jira/browse/PROTON-2275?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17752482#comment-17752482 ]
ASF GitHub Bot commented on PROTON-2275: ---------------------------------------- cpt-codes commented on PR #255: URL: https://github.com/apache/qpid-proton/pull/255#issuecomment-1671815671 Hi @gemmellr. I've come across the same issue using version 0.37.0 through vcpkg (happened to be the version I had installed, and I couldn't see any changes related to this in newer versions). I applied this little fix and it seems to work a treat. Any chance this could be reviewed soon? > idle_timeout connection option not working for SSL connections on Windows > ------------------------------------------------------------------------- > > Key: PROTON-2275 > URL: https://issues.apache.org/jira/browse/PROTON-2275 > Project: Qpid Proton > Issue Type: Bug > Components: proton-c > Environment: Windows client connecting to SSL broker > Reporter: Attila Kun > Priority: Major > > Steps to reproduce: > # Connect your receiver to an SSL-capable broker. The receiver has to be > running on Windows. > # Set the {{idle_timeout}} connection option to e.g. 2 seconds in your > {{proton::messaging_handler}} instance. > # 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|https://github.com/apache/qpid-proton/blob/0.31.0/c/src/ssl/schannel.c#L1850]. > After ensuring that schannel.c matches the behaviour of transport.c, the > problem went away. > Suggested fix: https://github.com/apache/qpid-proton/pull/255 -- This message was sent by Atlassian Jira (v8.20.10#820010) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org