Skip to content

Conversation

@sethmlarson
Copy link
Owner

No description provided.

@notatallshaw
Copy link

FYI, cffi needs to be installed from Github source for Python 3.13 until their next release, like in this PR: pypa/pip#12620

@sethmlarson
Copy link
Owner Author

@notatallshaw Thanks for the link, I'll give that a try :)

@sethmlarson sethmlarson requested a review from davisagli May 5, 2024 02:04
@sethmlarson
Copy link
Owner Author

@davisagli I believe what's happening is that we're seeing one type (bytes) from get_unverified_chain() for MemoryBIO and another type (_ssl.Certificate) from SSLSocket which is why I added the isinstance check.

Copy link
Collaborator

@davisagli davisagli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sethmlarson LGTM, but I wonder if that inconsistency should be reported upstream

@sethmlarson sethmlarson merged commit a418153 into main May 6, 2024
@sethmlarson sethmlarson deleted the test-other-oses branch May 6, 2024 13:09
@sethmlarson
Copy link
Owner Author

@davisagli Definitely, here's the upstream issue: python/cpython#118658

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.

4 participants