Skip to content

Change EthernetClient to use IANA recommended ephemeral port range #2345

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

Merged
merged 1 commit into from
Oct 8, 2014

Conversation

JChristensen
Copy link
Contributor

Certain ISPs block certain ports and this can cause connect failures for Ethernet clients. By changing the port range used in EthernetClient.cpp from 1024-65535 to the IANA-recommended "ephemeral port" range of 49152-65535, these failures should be minimized as ISPs should be less likely to block ports in this range. Note that recent versions of Windows and FreeBSD use the IANA-recommended range. See http://en.wikipedia.org/wiki/Ephemeral_port

In my case, failures occurred on port 1080, which is blocked by Comcast (one of the largest ISPs in the USA), see http://customer.comcast.com/help-and-support/internet/list-of-blocked-ports/

@matthijskooijman
Copy link
Collaborator

This is essentially a duplicate of #2249, but since this request has some more reasoning and uses a more standard port number, I'll close that one in favour of this one.

@matthijskooijman matthijskooijman added Library: Ethernet The Ethernet Arduino library feature request A request to make an enhancement (not a bug fix) labels Oct 8, 2014
cmaglie added a commit that referenced this pull request Oct 8, 2014
Change EthernetClient to use IANA recommended ephemeral port range
@cmaglie cmaglie merged commit d63350b into arduino:master Oct 8, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request A request to make an enhancement (not a bug fix) Library: Ethernet The Ethernet Arduino library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants