WebServer: Make it possible to create and use derived classes

[hurricanefrog]

Related area

WebServer

Hardware specification

Doesn't matter

Is your feature request related to a problem?

In the WebServer class, void WebServer::sendContent_P(PGM_P content, size_t size) is not virtual, i.e.. not overridable. However, its implementation uses _currentClient. That makes it impossible to implement a derived class, e.g. to implement a TLS-protected SecureWebServer. For that, I'd have to use a SecureClient member.

At least that applies when I want to do "proper polymorphism", i.e. access my potential SecureWebServer via a WebServer pointer.

Describe the solution you'd like

• Make all methods using _currentClient virtual
• Alternatively, make use of the client() virtual method consistently?

Describe alternatives you've considered

No response

Additional context

No response

I have checked existing list of Feature requests and the Contribution Guide

• I confirm I have checked existing list of Feature requests and Contribution Guide.

[me-no-dev]

Given that you have a use case and test case, I suggest you make the changes on your side and propose them as a pull request that we can review

[hurricanefrog]

Actually, now that I think about it, using client() instead of _currentClient is probably the quickest and best option.

Given that you have a use case and test case, I suggest you make the changes on your side and propose them as a pull request that we can review

Will do.