Add possibility to change server tcp port in void setup. No more fixed port!

[5a2v0]

Before this modification:

EthernetServer server(80);
void setup() {
//some stuff here
server.begin(); //User can't change initial port value
//other stuff here
}

Now:

EthernetServer server; //without port declaration but still working old declaration system...
void setup() {
//some stuff here
unsigned int tcp = EEPROM.read(x)
server.begin(tcp); //User can now use a different port, maybe saved in eprom directly from the sketch in a configuration page that permise to choice a value for the tcp port...
//other stuff here
}

Many thanks to Arduino's forum user: SukkoPera

[sandeepmistry]

Hi @5a2v0,

Thanks for submitting this.

I'm wondering if the following is a better way to achieve the same functionality without changing the API. I'm concerned about fragmenting the API, because other network libraries that extend Server (like WiFi101) would not have the change.

int port = 80; // default port
EthernetServer server(port);

void setup() {
  //some stuff here

  port = EEPROM.read(x);

  if (port > 0) {
    server = EthernetServer(port); // change the port
  }

  server.begin(); //User can now use a different port, maybe saved in eprom directly from the sketch in a configuration page that permise to choice a value for the tcp port...

  //other stuff here
}

[zoomx]

@sandeepmistry
maybe you mean server.begin(); instead of server.begin(tcp);?

[sandeepmistry]

@zoomx yes, sorry. I will update my previous comment.

[CLAassistant]

All committers have signed the CLA.

[JAndrassy]

In the newest WiFi library WiFiS3 for the Uno R4, Arduino has begin(port) and ctor without parameter, which I guess officially introduces it into the Arduino networking API.

overview of Server implementations in libraries https://github.com/JAndrassy/Arduino-Networking-API/blob/main/ArduinoNetAPILibs.md#server-class