Skip to content
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

Happy Eyeballs support in Socket.ConnectAsync #87932

Open
liveans opened this issue Jun 22, 2023 · 2 comments
Open

Happy Eyeballs support in Socket.ConnectAsync #87932

liveans opened this issue Jun 22, 2023 · 2 comments
Assignees
Labels
api-approved API was approved in API review, it can be implemented area-System.Net.Sockets
Milestone

Comments

@liveans
Copy link
Member

liveans commented Jun 22, 2023

This API has been proposed and approved on #861.
Since we're closing the mega-issue(#33418) we'll track this here.

class Socket
{
	// existing: public static bool ConnectAsync(SocketType socketType, ProtocolType protocolType, SocketAsyncEventArgs e);
	public static bool ConnectAsync (SocketType socketType, ProtocolType protocolType, SocketAsyncEventArgs e, ConnectAlgorithm connectAlgorithm);
}

// new enum
enum ConnectAlgorithm
{
    // use existing behavior.
    Default,

    // use a Happy Eyeballs-like algorithm to connect.
    Parallel = 1
}
@liveans liveans added api-approved API was approved in API review, it can be implemented area-System.Net.Sockets labels Jun 22, 2023
@ghost ghost added the untriaged New issue has not been triaged by the area owner label Jun 22, 2023
@ghost
Copy link

ghost commented Jun 22, 2023

Tagging subscribers to this area: @dotnet/ncl
See info in area-owners.md if you want to be subscribed.

Issue Details

This API has been proposed and approved on #861.
Since we're closing the mega-issue(#33418) we'll track this here.

class Socket
{
	// existing: public static bool ConnectAsync(SocketType socketType, ProtocolType protocolType, SocketAsyncEventArgs e);
	public static bool ConnectAsync (SocketType socketType, ProtocolType protocolType, SocketAsyncEventArgs e, ConnectAlgorithm connectAlgorithm);
}

// new enum
enum ConnectAlgorithm
{
    // use existing behavior.
    Default,

    // use a Happy Eyeballs-like algorithm to connect.
    Parallel = 1
}
Author: liveans
Assignees: -
Labels:

api-approved, area-System.Net.Sockets

Milestone: -

@liveans liveans added this to the Future milestone Jun 22, 2023
@ghost ghost removed the untriaged New issue has not been triaged by the area owner label Jun 22, 2023
@wfurt
Copy link
Member

wfurt commented Jun 23, 2023

related to #26177

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api-approved API was approved in API review, it can be implemented area-System.Net.Sockets
Projects
None yet
Development

No branches or pull requests

3 participants