Skip to content

gh-123174: Add proto parameter to sock_connect #123175

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Aperence
Copy link

@Aperence Aperence commented Aug 20, 2024

Allow the selection of the protocol used by getaddrinfo by adding an additional, optional parameter proto_addr_info.

If not specified, use the protocol used by the socket (= previous behaviour).

This change is needed to allow custom protocols to be used with sock_connect, because these protocols may raise an error when calling getaddrinfo (result in a ai_proto not supported error).
An example of such protocol is MPTCP (https://mptcp.dev)

@ghost
Copy link

ghost commented Aug 20, 2024

All commit authors signed the Contributor License Agreement.
CLA signed

@bedevere-app
Copy link

bedevere-app bot commented Aug 20, 2024

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.

Allow the selection of the protocol used by getaddrinfo by adding an additional,
optional parameter proto_addr_info.
If not specified, use the protocol used by the socket (= previous behaviour)
This change is needed to allow custom protocols to be used with the sock_connect, because these protocols
may raise an error when calling getaddrinfo (result in a "ai_proto not supported" error)
An example of such protocol is MPTCP (https://mptcp.dev)
@bedevere-app
Copy link

bedevere-app bot commented Aug 20, 2024

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.

Co-authored-by: Peter Bierma <zintensitydev@gmail.com>
@python-cla-bot
Copy link

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

CLA signed

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.

2 participants