GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — gh-161763: Correct SSL docs for unconnected client sockets by notatallshaw · Pull Request #161772 · python/cpython

Conversation

@notatallshaw

@notatallshaw notatallshaw commented Nov 10, 2026

Copy link
Copy Markdown
Contributor

The custom-context client example wraps an unconnected socket, so ssock.version() returns None. Connect the socket before printing the version, and clarify that connect() creates the underlying SSL object.

  • Issue: ssl docs: custom client socket example never connects #161763

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@notatallshaw
pythongh-161763: Correct SSL docs for unconnected client sockets
a86e02a
@notatallshaw
notatallshaw requested review from gpshead and picnixz as code owners November 10, 2026 18:29
@bedevere-app bedevere-app Bot mentioned this pull request Nov 10, 2026
ssl docs: custom client socket example never connects #161763
Open
@bedevere-app bedevere-app Bot added docs Documentation in the Doc dir skip news awaiting review labels Nov 10, 2026
@github-project-automation github-project-automation Bot added this to Docs PRs Nov 10, 2026
@github-project-automation github-project-automation Bot moved this to Todo in Docs PRs Nov 10, 2026
@notatallshaw notatallshaw mentioned this pull request Nov 10, 2026
Fix trusted connections for sockets wrapped before connect sethmlarson/truststore#271
Open
@read-the-docs-community

read-the-docs-community Bot commented Nov 10, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

Documentation build overview

📚 cpython-previews | 🛠️ Build #35793382 | 📁 Comparing c883649 against main (44ff6c9)

  🔍 Preview build  

1 file changed
± library/ssl.html
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@notatallshaw
pythongh-161763: Qualify SSL socket method references
c883649
gpshead
gpshead approved these changes Nov 12, 2026
View reviewed changes
@bedevere-app bedevere-app Bot added awaiting merge and removed awaiting review labels Nov 12, 2026
@gpshead gpshead added needs backport to 3.14 bugs and security fixes needs backport to 3.15 bugs and security fixes labels Nov 12, 2026
@gpshead
gpshead enabled auto-merge (squash) November 12, 2026 00:01
@gpshead
gpshead merged commit f112215 into python:main Nov 12, 2026
38 checks passed

Uh oh!

There was an error while loading. Please reload this page.
@github-project-automation github-project-automation Bot moved this from Todo to Done in Docs PRs Nov 12, 2026
@miss-islington-app

miss-islington-app Bot commented Nov 12, 2026

Copy link
Copy Markdown

Thanks @notatallshaw for the PR, and @gpshead for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14, 3.15.
🐍🍒⛏🤖
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@bedevere-app bedevere-app Bot removed the awaiting merge label Nov 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gpshead gpshead gpshead approved these changes
@picnixz picnixz Awaiting requested review from picnixz picnixz is a code owner

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

docs Documentation in the Doc dir needs backport to 3.14 bugs and security fixes needs backport to 3.15 bugs and security fixes skip news

Projects

Docs PRs
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@notatallshaw @gpshead
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.