Skip to content

Conversation

@bneradt
Copy link
Contributor

@bneradt bneradt commented Aug 17, 2023

ssl.wrap_socket was deprecated in Python 3.6 and is removed in Python 3.12. This patch replaces it with the recommended SSLContext.wrap_socket method instead. This fixes the extension for Python 3.12.

ssl.wrap_socket was deprecated in Python 3.6 and is removed in Python
3.12. This patch replaces it with the recommended SSLContext.wrap_socket
method instead. This fixes the extension for Python 3.12.
@bneradt bneradt added the AuTest label Aug 17, 2023
@bneradt bneradt added this to the 10.0.0 milestone Aug 17, 2023
@bneradt bneradt requested a review from maskit August 17, 2023 22:50
@bneradt bneradt self-assigned this Aug 17, 2023
Copy link
Member

@maskit maskit left a comment

Choose a reason for hiding this comment

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

Looks good, though I don't know Python very much.

Should we back port this to 9.2.x? The new way has been there for years so it should be possible.

@bneradt
Copy link
Contributor Author

bneradt commented Aug 18, 2023

Looks good, though I don't know Python very much.

Should we back port this to 9.2.x? The new way has been there for years so it should be possible.

Yes, this is wise. We'll want to run CI on 9.2.x branch releases not infrequently.

@bneradt bneradt merged commit 5d4f975 into apache:master Aug 18, 2023
@bneradt bneradt deleted the fix_microserver_ext_for_python3_12 branch August 18, 2023 02:55
zwoop pushed a commit that referenced this pull request Aug 28, 2023
ssl.wrap_socket was deprecated in Python 3.6 and is removed in Python
3.12. This patch replaces it with the recommended SSLContext.wrap_socket
method instead. This fixes the extension for Python 3.12.

(cherry picked from commit 5d4f975)
@zwoop
Copy link
Contributor

zwoop commented Aug 28, 2023

Cherry-picked to v9.2.x

@zwoop zwoop modified the milestones: 10.0.0, 9.2.3 Aug 28, 2023
cmcfarlen pushed a commit to cmcfarlen/trafficserver that referenced this pull request Jun 3, 2024
* asf/master:
  Upgrade yaml-cpp version to 0.8.0 (apache#10249)
  This dependency is not needed, cmake did it right (apache#10250)
  Revert "Make OSX and FreeBSD not required temporarily (apache#10237)" (apache#10248)
  Python 3.12: microserver.test.ext wrap_socket update (apache#10247)
  Coverity 1508984: Dereference null return value (apache#10245)
  fall back to configure file for older cmake versions (apache#10236)
  TLS early data: logging updates (apache#10115)
  Fixing TSHttpTxnServerAddrSet (apache#10189)
  Make OSX and FreeBSD not required temporarily (apache#10237)
  Fixes a problem which can decrement milestone metrics unintentionally (apache#10188)
  Fix editor config for makefiles. (apache#10190)
  Correctly handle encoding for cache hash generation (apache#10126)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants