[jira] [Commented] (TS-4412) event overwrited by SSL_READ_READY / COMPLETE in ssl_read_from_net

["Susan Hinrichs (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/TS-4412?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15316641#comment-15316641 ] 

Susan Hinrichs commented on TS-4412:
------------------------------------

[~cschombu] reported on the mailing list openssl problems when moving from 4.x to 6.0.  Sounds like his traffic was potentially over-reading and then losing data.  I plan on putting up a PR which adds a ntodo check on the read loop.  He should be able to back port that to 6.0 and see if that helps his case.

> event overwrited by SSL_READ_READY / COMPLETE in ssl_read_from_net
> ------------------------------------------------------------------
>
>                 Key: TS-4412
>                 URL: https://issues.apache.org/jira/browse/TS-4412
>             Project: Traffic Server
>          Issue Type: Bug
>          Components: SSL
>            Reporter: Oknet Xu
>            Assignee: Susan Hinrichs
>             Fix For: 7.0.0
>
>
> the {{ssl_read_from_net}} read data from socket and saved into MIOBuffer.
> It was designed to fill the MIOBuffer as much as possible, even execeed the VIO {{nbytes}} setting.
> It will try to call {{SSLReadBuffer}} again and again until the MIOBuffer is full or an error returned from {{SSLReadBuffer}}. The {{event}} will be set to a error status if an error returned from {{SSLReadBuffer}}.
> But at the end of {{ssl_read_from_net}}, the {{event}} will be overwrited to SSL_READ_READY or SSL_READ_COMPLETE depend on {{vio.ntodo()}} if this is one success at least in {{SSLReadBuffer}} before an error.
> For example:
>     The {{SSLNetVConnection::net_read_io}} could not get SSL_READ_EOS from {{ssl_read_from_net}} and can not close the SSLNetVConnection immediately. Because The {{event}} SSL_READ_EOS is overwrited.
>     The SSLNetVConnection will be closed by timeout.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)