GitHub user shinrich opened a pull request:

    https://github.com/apache/trafficserver/pull/1526

    When SSL connect fails, we return 502 success

    While debugging a failure in origin cert verification, we saw that the 
response header included
    
    ```
    HTTP/1.1 502 Success
    Date: Mon, 27 Feb 2017 15:18:29 GMT
    Connection: keep-alive
    ```
    502 Success seemed peculiar.  Tracked it down to the fact that errno was 
not set in the SSL_connect failure case.  Added a check to stuff in 
ENET_CONNECT_FAILED if errno is 0 so we get a reason that is in the ballpark.
    


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/shinrich/trafficserver success_on_502_pr

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/trafficserver/pull/1526.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1526
    
----
commit 085a5983b706a875229a2ac03d76174b9c79a2f5
Author: Susan Hinrichs <shinr...@ieee.org>
Date:   2017-03-01T21:33:23Z

    When SSL connect fails, we return 502 success

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to