ci: revert accidental max_args_count/_bytes member additions

My earlier 2e803ed (fd_set/socklen_t CI fix) accidentally swept in
two const std::size_t members (max_args_count, max_args_bytes) that
were sitting uncommitted in the working tree as part of an unrelated
in-progress feature. The matching create_webserver setters / private
fields were not in HEAD, so the constructor's params._max_args_count
referenced a nonexistent member.

Walk back the const-member additions in both webserver.hpp and the
constructor init in webserver.cpp so HEAD is internally consistent and
the CI fix in 2e803ed (fd_set include + dropped socklen_t signedness
assert) stands alone. The full max_args feature stays in the working
tree where it was, to be committed as one coherent unit later.

Author: etr

src/httpserver/webserver.hpp

@@ -411,13 +411,6 @@ class webserver {
      const struct sockaddr* bind_address;
      std::shared_ptr<struct sockaddr_storage> bind_address_storage;
      const int max_thread_stack_size;
-     // 0 sentinel -> compile-time defaults
-     // (arguments_accumulator::DEFAULT_MAX_ARGS_COUNT / _BYTES).
-     // webserver_impl::connection_notify copies these into the per-
-     // connection_state at MHD_CONNECTION_NOTIFY_STARTED; populate_args
-     // reads them from connection_state via the socket_context.
-     const std::size_t max_args_count;
-     const std::size_t max_args_bytes;
      const bool use_ssl;
      const bool use_ipv6;
      const bool use_dual_stack;

src/webserver.cpp

@@ -200,8 +200,6 @@ webserver::webserver(const create_webserver& params):
     bind_address(params._bind_address),
     bind_address_storage(params._bind_address_storage),
     max_thread_stack_size(params._max_thread_stack_size),
-    max_args_count(params._max_args_count),
-    max_args_bytes(params._max_args_bytes),
     use_ssl(params._use_ssl),
     use_ipv6(params._use_ipv6),
     use_dual_stack(params._use_dual_stack),