refactor: drop HTTP_ALL, use HTTP_ANY (255) from HTTP_Method.h as the match-any sentinel

- Include <HTTP_Method.h> first (defines HTTPMethod and HTTP_ANY=(HTTPMethod)(255));
  fall back to <http_parser.h> then inline fallback, both of which also define
  HTTP_ANY ((http_method)(255)) matching Arduino's value
- Remove #define HTTP_ALL entirely - users use HTTP_ANY from the platform
- WebRequestMethodComposite(WebRequestMethod m): treat HTTP_ANY (value 255) as
  "match any" sentinel by creating an empty composite (same as default ctor)
- Update on() default and all docs from HTTP_ALL -> HTTP_ANY

Co-authored-by: mathieucarbou <61346+mathieucarbou@users.noreply.github.com>

Author: Copilot

docs/backup/wiki.md

@@ -1887,7 +1887,7 @@ void setup(){
   }, onUpload);
 
   // send a file when /index is requested
-  server.on("/index", HTTP_ALL, [](AsyncWebServerRequest *request){
+  server.on("/index", HTTP_ANY, [](AsyncWebServerRequest *request){
     request->send(SPIFFS, "/index.htm");
   });
 
@@ -1974,10 +1974,10 @@ public :
 
   void begin(){
     // attach global request handler
-    classWebServer.on("/example", HTTP_ALL, handleRequest);
+    classWebServer.on("/example", HTTP_ANY, handleRequest);
 
     // attach class request handler
-    classWebServer.on("/example", HTTP_ALL, std::bind(&WebClass::classRequest, this, std::placeholders::_1));
+    classWebServer.on("/example", HTTP_ANY, std::bind(&WebClass::classRequest, this, std::placeholders::_1));
   }
 };
 
@@ -1986,10 +1986,10 @@ WebClass webClassInstance;
 
 void setup() {
   // attach global request handler
-  globalWebServer.on("/example", HTTP_ALL, handleRequest);
+  globalWebServer.on("/example", HTTP_ANY, handleRequest);
 
   // attach class request handler
-  globalWebServer.on("/example", HTTP_ALL, std::bind(&WebClass::classRequest, webClassInstance, std::placeholders::_1));
+  globalWebServer.on("/example", HTTP_ANY, std::bind(&WebClass::classRequest, webClassInstance, std::placeholders::_1));
 }
 
 void loop() {

docs/setup.md

@@ -71,12 +71,12 @@ void setup(){
   }, onUpload);
 
   // send a file when /index is requested (SPIFFS example)
-  server.on("/index", HTTP_ALL, [](AsyncWebServerRequest *request){
+  server.on("/index", HTTP_ANY, [](AsyncWebServerRequest *request){
     request->send(SPIFFS, "/index.htm");
   });
 
   // send a file when /index is requested (LittleFS example)
-  server.on("/index", HTTP_ALL, [](AsyncWebServerRequest *request){
+  server.on("/index", HTTP_ANY, [](AsyncWebServerRequest *request){
     request->send(LittleFS, "/index.htm");
   });
 
@@ -161,10 +161,10 @@ public :
 
   void begin(){
     // attach global request handler
-    classWebServer.on("/example", HTTP_ALL, handleRequest);
+    classWebServer.on("/example", HTTP_ANY, handleRequest);
 
     // attach class request handler
-    classWebServer.on("/example", HTTP_ALL, std::bind(&WebClass::classRequest, this, std::placeholders::_1));
+    classWebServer.on("/example", HTTP_ANY, std::bind(&WebClass::classRequest, this, std::placeholders::_1));
   }
 };
 
@@ -173,10 +173,10 @@ WebClass webClassInstance;
 
 void setup() {
   // attach global request handler
-  globalWebServer.on("/example", HTTP_ALL, handleRequest);
+  globalWebServer.on("/example", HTTP_ANY, handleRequest);
 
   // attach class request handler
-  globalWebServer.on("/example", HTTP_ALL, std::bind(&WebClass::classRequest, webClassInstance, std::placeholders::_1));
+  globalWebServer.on("/example", HTTP_ANY, std::bind(&WebClass::classRequest, webClassInstance, std::placeholders::_1));
 }
 
 void loop() {

src/ESPAsyncWebServer.h

@@ -45,16 +45,25 @@
 #error Platform not supported
 #endif
 
-// HTTP method enum from the underlying http_parser library.
-// All supported TCP libraries (AsyncTCP, ESPAsyncTCP, RPAsyncTCP) are built on
-// http_parser, so http_method is available on all supported platforms.
-// Using the platform's enum avoids redefining HTTP_GET, HTTP_POST, etc. ourselves,
-// which eliminates name conflicts with other libraries (e.g., Arduino WebServer.h).
-#if __has_include(<http_parser.h>)
+// HTTP method types from the platform's HTTP parser library.
+// Arduino ESP32 core provides HTTP_Method.h which typedef's http_method as
+// HTTPMethod and defines HTTP_ANY = (HTTPMethod)(255) as the "match any" sentinel.
+// Fall back to the raw http_parser.h (always available via the TCP library) or,
+// as a last resort, to a fully inline fallback definition.
+#if __has_include(<HTTP_Method.h>)
+#  include <HTTP_Method.h>
+// HTTP_Method.h provides: typedef enum http_method HTTPMethod;
+// and: #define HTTP_ANY (HTTPMethod)(255)
+#elif __has_include(<http_parser.h>)
 #  include <http_parser.h>
+// http_parser.h provides enum http_method but not the HTTP_ANY sentinel.
+// Define the sentinel here, matching the value used by Arduino's HTTP_Method.h.
+#  ifndef HTTP_ANY
+#    define HTTP_ANY ((http_method)(255))
+#  endif
 #else
-// Fallback definition matching the llhttp/http_parser spec (used by all platforms).
-// This is provided only for toolchains that do not expose http_parser.h directly.
+// Full fallback for toolchains that expose neither header.
+// Enum values match the llhttp/http_parser spec used by all supported platforms.
 typedef enum {
   HTTP_DELETE = 0, HTTP_GET = 1, HTTP_HEAD = 2, HTTP_POST = 3, HTTP_PUT = 4,
   HTTP_CONNECT = 5, HTTP_OPTIONS = 6, HTTP_TRACE = 7, HTTP_COPY = 8, HTTP_LOCK = 9,
@@ -65,6 +74,7 @@ typedef enum {
   HTTP_SUBSCRIBE = 26, HTTP_UNSUBSCRIBE = 27, HTTP_PATCH = 28, HTTP_PURGE = 29,
   HTTP_MKCALENDAR = 30, HTTP_LINK = 31, HTTP_UNLINK = 32,
 } http_method;
+#  define HTTP_ANY ((http_method)(255))
 #endif
 
 #include "AsyncWebServerVersion.h"
@@ -104,28 +114,35 @@ class AsyncMiddlewareChain;
 // platform's http_parser enum.  HTTP_GET, HTTP_POST, HTTP_DELETE, HTTP_PUT,
 // HTTP_PATCH, HTTP_HEAD, HTTP_OPTIONS, HTTP_PROPFIND, HTTP_LOCK, HTTP_UNLOCK,
 // HTTP_PROPPATCH, HTTP_MKCOL, HTTP_MOVE, HTTP_COPY and many others are already
-// defined globally by <http_parser.h> above — no redefinition needed here.
+// defined globally by the platform headers above — no redefinition needed here.
+// HTTP_ANY (= 255) is the "match any method" sentinel, also from the platform.
 typedef http_method WebRequestMethod;
 
 // WebRequestMethodComposite: an ordered list of HTTP methods that a handler
-// accepts.  An empty composite matches *any* method (HTTP_ANY / HTTP_ALL
-// semantics).  Using a list avoids bitmask arithmetic on the platform enum's
+// accepts.  An empty composite (or one that contains HTTP_ANY) matches *any*
+// method.  Using a list avoids bitmask arithmetic on the platform enum's
 // non-power-of-two sequential values.
 class WebRequestMethodComposite {
 public:
   // Maximum number of methods that can be listed.  In practice, handlers
-  // accept at most a handful of methods; use the empty composite (HTTP_ALL)
-  // when you want to match any method.
+  // accept at most a handful of methods; use HTTP_ANY when you want to
+  // match any method.
   static const uint8_t MAX_METHODS = 8;
 
   // Empty composite = match any method.
   WebRequestMethodComposite() : _count(0) {}
 
   // Single-method composite.  Intentionally implicit so that a bare
   // WebRequestMethod (e.g. HTTP_GET) is accepted wherever a composite is
-  // expected.
-  WebRequestMethodComposite(WebRequestMethod m) : _count(1) {
-    _methods[0] = m;
+  // expected.  HTTP_ANY (value 255) is treated as the "match any" sentinel:
+  // it creates an empty composite rather than a single-method one.
+  WebRequestMethodComposite(WebRequestMethod m) {
+    if (m == static_cast<WebRequestMethod>(HTTP_ANY)) {
+      _count = 0;  // HTTP_ANY: match any method
+    } else {
+      _count = 1;
+      _methods[0] = m;
+    }
   }
 
   // Append a method; silently ignores the call once MAX_METHODS is reached.
@@ -179,17 +196,6 @@ inline bool operator^(const WebRequestMethodComposite &c, WebRequestMethod m) {
   return c.contains(m);
 }
 
-// HTTP_ALL: empty composite — matches any HTTP method.
-// HTTP_ANY: backward-compatible alias for HTTP_ALL.
-// HTTP_ANY is only defined here when external headers have not already defined
-// it as a macro (e.g., Arduino core may define HTTP_ANY as an integer).  When
-// an external macro is present the name remains an integer; user code should
-// migrate to HTTP_ALL which is always safe.
-#define HTTP_ALL WebRequestMethodComposite()
-#ifndef HTTP_ANY
-#define HTTP_ANY WebRequestMethodComposite()
-#endif
-
 #ifndef HAVE_FS_FILE_OPEN_MODE
 namespace fs {
 class FileOpenMode {
@@ -1629,7 +1635,7 @@ class AsyncWebServer : public AsyncMiddlewareChain {
   bool removeHandler(AsyncWebHandler *handler);
 
   AsyncCallbackWebHandler &on(AsyncURIMatcher uri, ArRequestHandlerFunction onRequest) {
-    return on(std::move(uri), HTTP_ALL, onRequest);
+    return on(std::move(uri), HTTP_ANY, onRequest);
   }
   AsyncCallbackWebHandler &on(
     AsyncURIMatcher uri, WebRequestMethodComposite method, ArRequestHandlerFunction onRequest, ArUploadHandlerFunction onUpload = nullptr,