Skip to content

Conversation

@mckirk
Copy link

@mckirk mckirk commented Jul 7, 2025

This should fix #179.

@sethmlarson
Copy link
Owner

I don't love that we're going to have to do this for many other libraries :/ Boto doesn't support a way to configure SSLContexts?

@mckirk
Copy link
Author

mckirk commented Jul 8, 2025

I don't love that we're going to have to do this for many other libraries :/ Boto doesn't support a way to configure SSLContexts?

Yeah, it's admittedly not so cool that these hacks are apparently necessary... but botocore even seems to go out of its way to get an 'unpatched' SSLContext: https://github.com/boto/botocore/blob/develop/botocore/httpsession.py#L40

But maybe they'd be open to removing that, the commit that introduced this is seven years old:
boto/botocore@9ce22db

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.

RecursionError: maximum recursion depth exceeded

2 participants