WebServer: Use client() instead of _currentClient #11945
Conversation
|
|
👋 Hello hurricanefrog, we appreciate your contribution to this project! 📘 Please review the project's Contributions Guide for key guidelines on code, documentation, testing, and more. 🖊️ Please also make sure you have read and signed the Contributor License Agreement for this project. Click to see more instructions ...
Review and merge process you can expect ...
|
hurricanefrog
force-pushed
the
issue/11944-derivable-webserver
branch
from
8cf7d81 to
aa8c053
Compare
|
Should I really lowercase the WebServer component to |
It's not a big deal, but it's nice to have consistency. Will not influence wether this gets merged or or not :) |
this makes it possible to create derived classes properly.
hurricanefrog
force-pushed
the
issue/11944-derivable-webserver
branch
from
aa8c053 to
c80e972
Compare
Lets keep this open then (I'll convert it to Draft) until you can test it, in case other changes also need to be added. Please let us know when you have done that :) |
Test Results 76 files 76 suites 15m 57s ⏱️ Results for commit c80e972. |
Memory usage test (comparing PR against master branch)The table below shows the summary of memory usage change (decrease - increase) in bytes and percentage for each target.
Click to expand the detailed deltas report [usage change in BYTES]
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
3 participants
This makes it possible to create derived classes properly.
Description of Change
This change replaces all uses of
_currentClientin theWebServerclass by calling the virtual methodclient().(Except in
handleClient, which makes to sense to touch, I think).This makes it possible to create derived classes, e.g to create TLS-protected web servers.
This was not possible previously due to the hardcoded "plain"
_currentClient.Derived classes can now simply override
client(), e.g. to return the TLS protected socket, and continue using the rest of the class (i.e. all of the HTTP parsing).Test Scenarios
I haven't done any tests. I expect no impact on the function of the code. I haven't been able to create a HTTPS server with the changes yet, to confirm it leads to my goal, this takes some additional time.
Related links
closes #11944