Skip to content

gh-146139: Disable socketpair authentication on WASI#146140

Open
dicej wants to merge 2 commits intopython:mainfrom
dicej:disable-sockepair-auth-on-wasi
Open

gh-146139: Disable socketpair authentication on WASI#146140
dicej wants to merge 2 commits intopython:mainfrom
dicej:disable-sockepair-auth-on-wasi

Conversation

@dicej
Copy link
Copy Markdown

@dicej dicej commented Mar 18, 2026

Calling connect(2) on a non-blocking socket on WASI may leave the socket in a "connecting" but not yet "connected" state. In the former case, calling getpeername(2) on it will fail, leading to an unhandled exception in Python.

Calling `connect(2)` on a non-blocking socket on WASI may leave the socket in a
"connecting" but not yet "connected" state.  In the former case, calling
`getpeername(2)` on it will fail, leading to an unhandled exception in Python.
@bedevere-app
Copy link
Copy Markdown

bedevere-app bot commented Mar 18, 2026

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@python-cla-bot
Copy link
Copy Markdown

The following commit authors need to sign the Contributor License Agreement:

CLA not signed

Copy link
Copy Markdown
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

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

LGTM

@vstinner
Copy link
Copy Markdown
Member

@dicej: You should sign the Python CLA: #146140 (comment).

Co-authored-by: Victor Stinner <vstinner@python.org>
@bedevere-app
Copy link
Copy Markdown

bedevere-app bot commented Mar 19, 2026

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@dicej
Copy link
Copy Markdown
Author

dicej commented Mar 19, 2026

Regarding the CLA: I've asked my employer for permission to sign it and am awaiting a reply.

@vstinner
Copy link
Copy Markdown
Member

Regarding the CLA: I've asked my employer for permission to sign it and am awaiting a reply.

Any update on the CLA @dicej?

@dicej
Copy link
Copy Markdown
Author

dicej commented Mar 24, 2026

Any update on the CLA @dicej?

Yes: I need my employer to sign it as an organization to cover my contributions (i.e. I'm not supposed to sign as an individual contributor), which means using this form, but it's not clear how we should fill that out given that the signatory (i.e. the person with authority to sign for the whole organization) and the contributor(s) (e.g. me) will be different people. I sent an email to psf@python.org for clarification and was redirected to contributors@python.org, but haven't heard back yet.

Given this is effectively a one-line change modulo whitespace, it all feels like overkill, but this should unblock future contributions as well.

@brettcannon
Copy link
Copy Markdown
Member

it's not clear how we should fill that out given that the signatory (i.e. the person with authority to sign for the whole organization) and the contributor(s) (e.g. me) will be different people.

It's fine if they are different people. The key thing is if someone from Akamai is submitting for the entire org then they just need to be empowered to sign such a form to say any contributions coming from someone on work time is covered by the CLA.

@dicej
Copy link
Copy Markdown
Author

dicej commented Mar 27, 2026

I still haven't received a response from contributors@python.org. Can anyone confirm that, if an authorized party at Akamai signs the CLA form on behalf of Akamai, any contributions from a @akamai.com email address will be covered?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants