Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

agent: return error when cert auth fails #22

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

zimbabao
Copy link

@zimbabao zimbabao commented Jul 3, 2017

Sites without intermediate certs in chain
and other ssl error ends up with sockets.ssl
being null and that causes Type error in agent.

Sites without intermediate certs in chain
and other ssl error ends up with sockets.ssl
being null and that causes Type error in agent.
@zimbabao zimbabao force-pushed the handle-ssl-authorization-error branch from 84d1b1a to ac936b1 Compare July 3, 2017 22:39
@dawsonc623
Copy link

@indutny It seems like this project has been abandoned, but my organization has been using it for the last couple of years. This issue is preventing us from a new release of one of our products. If the lint errors are fixed, could we get this merged and a patch release out?

If not, we can fork the project and fix it up ourselves, but I would rather continue to use and contribute to the original, open source version here if possible.

@indutny
Copy link
Owner

indutny commented Jul 10, 2019

@dawsonc623 could you send me an email (it is my github profile). I'd be happy to discuss this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants