Tighten QUERY: opt-in JsonHandler, HTTP_PARSER_HAS_QUERY mapping

Do not enable QUERY on AsyncCallbackJsonWebHandler by default. Map the platform http_method enumerator only when HTTP_PARSER_HAS_QUERY is defined (enum members are not preprocessor macros). Add Accept-Query and a Json example curl.

Author: EyJunge1

docs/requests.md

@@ -14,13 +14,19 @@ request->contentLength(); // size_t:  ContentLength of the request (not availabl
 request->multipart();     // bool:    True if the request has content type "multipart"
 ```
 
-`HTTP_QUERY` is [RFC 10008](https://www.rfc-editor.org/rfc/rfc10008.html): a safe, idempotent method that carries the query in the request body (same body path as POST/PUT/PATCH whenever `Content-Length` or chunked transfer is present). `AsyncCallbackJsonWebHandler` accepts QUERY with `Content-Type: application/json`. Unknown methods still abort the connection.
+`HTTP_QUERY` is [RFC 10008](https://www.rfc-editor.org/rfc/rfc10008.html): a safe, idempotent method that carries the query in the request body (same body path as POST/PUT/PATCH whenever `Content-Length` or chunked transfer is present). Unknown methods still abort the connection.
+
+`AsyncCallbackJsonWebHandler` does **not** enable QUERY by default. Call `setMethod(HTTP_QUERY)` (alone or combined with POST/PUT). QUERY still requires `Content-Type: application/json` (or MessagePack). Advertise supported query media types with `Accept-Query` (`asyncsrv::T_Accept_Query`), e.g. `application/json`.
+
+Platform `http_method` mapping is compiled in only when the platform parser defines `HTTP_PARSER_HAS_QUERY` (ESP-IDF after QUERY support). The library parser always recognizes `QUERY`.
 
 Register it like any other verb:
 
 ```cpp
 server.on("/search", HTTP_QUERY, [](AsyncWebServerRequest *request) {
-  request->send(200, "text/plain", "ok");
+  AsyncWebServerResponse *res = request->beginResponse(200, "text/plain", "ok");
+  res->addHeader("Accept-Query", "application/json");
+  request->send(res);
 });
 
 auto *json = new AsyncCallbackJsonWebHandler("/search", [](AsyncWebServerRequest *request, JsonVariant &json) {

examples/arduino/Json/Json.ino

@@ -63,6 +63,7 @@ void setup() {
 
   // curl -v -X POST -H 'Content-Type: application/json' -d '{"name":"You"}' http://192.168.4.1/json2
   // curl -v -X PUT -H 'Content-Type: application/json' -d '{"name":"You"}' http://192.168.4.1/json2
+  // curl -v -X QUERY -H 'Content-Type: application/json' -d '{"q":"You"}' http://192.168.4.1/json2
   //
   // edge cases:
   //
@@ -74,7 +75,7 @@ void setup() {
   // curl -v -X POST -H "Content-Type: application/json" -d "123456789" -H "Content-Length: 8" http://192.168.4.1/json2 => 12345678
   // curl -v -X POST -H "Content-Type: application/json" -d "123456789" -H "Content-Length: 9" http://192.168.4.1/json2 => 413: Content length exceeds maximum allowed
   handler->setMaxContentLength(8);
-  handler->setMethod(HTTP_POST | HTTP_PUT);
+  handler->setMethod(HTTP_POST | HTTP_PUT | HTTP_QUERY);
   handler->onRequest([](AsyncWebServerRequest *request, JsonVariant &json) {
     serializeJson(json, Serial);
     Serial.println();

src/AsyncJson.cpp

@@ -113,8 +113,7 @@ size_t AsyncMessagePackResponse::_fillBuffer(uint8_t *data, size_t len) {
 
 // Body handler supporting both content types: JSON and MessagePack
 constexpr static WebRequestMethodComposite JsonHandlerMethods =
-  AsyncWebRequestMethod::HTTP_GET | AsyncWebRequestMethod::HTTP_POST | AsyncWebRequestMethod::HTTP_PUT | AsyncWebRequestMethod::HTTP_PATCH
-  | AsyncWebRequestMethod::HTTP_QUERY;
+  AsyncWebRequestMethod::HTTP_GET | AsyncWebRequestMethod::HTTP_POST | AsyncWebRequestMethod::HTTP_PUT | AsyncWebRequestMethod::HTTP_PATCH;
 
 #if ARDUINOJSON_VERSION_MAJOR == 6
 AsyncCallbackJsonWebHandler::AsyncCallbackJsonWebHandler(AsyncURIMatcher uri, ArJsonRequestHandlerFunction onRequest, size_t maxJsonBufferSize)

src/ESPAsyncWebServer.h

@@ -230,7 +230,7 @@ class WebRequestMethodComposite {
       : MAP_EXTERNAL_TERNARY(HTTP_PURGE)
       : MAP_EXTERNAL_TERNARY(HTTP_LINK)
       : MAP_EXTERNAL_TERNARY(HTTP_UNLINK)
-#if defined(HTTP_QUERY)
+#if defined(HTTP_PARSER_HAS_QUERY)
       : MAP_EXTERNAL_TERNARY(HTTP_QUERY)
 #endif
 #if defined(HTTP_ANY)

src/literals.h

@@ -36,6 +36,7 @@ static constexpr const char T__opaque[] = "\", opaque=\"";
 static constexpr const char T_100_CONTINUE[] = "100-continue";
 static constexpr const char T_13[] = "13";
 static constexpr const char T_ACCEPT[] = "Accept";
+static constexpr const char T_Accept_Query[] = "Accept-Query";
 static constexpr const char T_Accept_Ranges[] = "Accept-Ranges";
 static constexpr const char T_attachment[] = "attachment; filename=\"";
 static constexpr const char T_AUTH[] = "Authorization";
@@ -244,9 +245,9 @@ DECLARE_STR(T_HTTP_CODE_507, "Insufficient Storage");
 DECLARE_STR(T_HTTP_CODE_ANY, "Unknown code");
 
 static constexpr const char *T_only_once_headers[] = {
-  T_Accept_Ranges,     T_Content_Length,   T_Content_Type, T_Connection, T_CORS_ACAC, T_CORS_ACAH,     T_CORS_ACAM, T_CORS_ACAO,
-  T_CORS_ACMA,         T_CORS_O,           T_Date,         T_DIGEST,     T_ETag,      T_Last_Modified, T_LOCATION,  T_retry_after,
-  T_Transfer_Encoding, T_Content_Location, T_Server,       T_WWW_AUTH
+  T_Accept_Ranges,     T_Accept_Query,     T_Content_Length, T_Content_Type, T_Connection, T_CORS_ACAC, T_CORS_ACAH,     T_CORS_ACAM,
+  T_CORS_ACAO,         T_CORS_ACMA,        T_CORS_O,         T_Date,         T_DIGEST,     T_ETag,      T_Last_Modified, T_LOCATION,
+  T_retry_after,       T_Transfer_Encoding, T_Content_Location, T_Server, T_WWW_AUTH
 };
 static constexpr size_t T_only_once_headers_len = sizeof(T_only_once_headers) / sizeof(T_only_once_headers[0]);
 static constexpr size_t T__GZ_LEN = sizeof(T__gz) - 1;