Introduce request#isMethod(...) to check the http method of the request (fixes #398)

Author: mathieucarbou

src/AsyncJson.cpp

@@ -115,15 +115,19 @@ size_t AsyncMessagePackResponse::_fillBuffer(uint8_t *data, size_t len) {
 
 #if ARDUINOJSON_VERSION_MAJOR == 6
 AsyncCallbackJsonWebHandler::AsyncCallbackJsonWebHandler(AsyncURIMatcher uri, ArJsonRequestHandlerFunction onRequest, size_t maxJsonBufferSize)
-  : _uri(std::move(uri)), _method(HTTP_GET | HTTP_POST | HTTP_PUT | HTTP_PATCH), _onRequest(onRequest), maxJsonBufferSize(maxJsonBufferSize),
-    _maxContentLength(16384) {}
+  : _uri(std::move(uri)), _method(HTTP_ANY), _onRequest(onRequest), maxJsonBufferSize(maxJsonBufferSize), _maxContentLength(16384) {}
 #else
 AsyncCallbackJsonWebHandler::AsyncCallbackJsonWebHandler(AsyncURIMatcher uri, ArJsonRequestHandlerFunction onRequest)
-  : _uri(std::move(uri)), _method(HTTP_GET | HTTP_POST | HTTP_PUT | HTTP_PATCH), _onRequest(onRequest), _maxContentLength(16384) {}
+  : _uri(std::move(uri)), _method(HTTP_ANY), _onRequest(onRequest), _maxContentLength(16384) {}
 #endif
 
 bool AsyncCallbackJsonWebHandler::canHandle(AsyncWebServerRequest *request) const {
-  if (!_onRequest || !request->isHTTP() || !(_method & request->method())) {
+  if (!_onRequest || !request->isHTTP()) {
+    return false;
+  }
+
+  // HTTP_GET | HTTP_POST | HTTP_PUT | HTTP_PATCH
+  if (!request->isMethod(HTTP_GET) && !request->isMethod(HTTP_POST) && !request->isMethod(HTTP_PUT) && !request->isMethod(HTTP_PATCH)) {
     return false;
   }
 

src/ESPAsyncWebServer.h

@@ -347,6 +347,13 @@ class AsyncWebServerRequest {
   WebRequestMethodComposite method() const {
     return _method;
   }
+  bool isMethod(WebRequestMethodComposite method) const {
+#if defined(TARGET_RP2040) || defined(TARGET_RP2350) || defined(PICO_RP2040) || defined(PICO_RP2350)
+    return (_method == method) || (method == HTTP_ANY);
+#else
+    return (_method & method) != 0;
+#endif
+  }
   const String &url() const {
     return _url;
   }

src/WebHandlers.cpp

@@ -300,7 +300,7 @@ void AsyncCallbackWebHandler::setUri(AsyncURIMatcher uri) {
 }
 
 bool AsyncCallbackWebHandler::canHandle(AsyncWebServerRequest *request) const {
-  if (!_onRequest || !request->isHTTP() || !(_method & request->method())) {
+  if (!_onRequest || !request->isHTTP() || !request->isMethod(_method)) {
     return false;
   }
   return _uri.matches(request);

src/WebRequest.cpp

@@ -1314,46 +1314,46 @@ const char *AsyncWebServerRequest::methodToString() const {
   if (_method == HTTP_ANY) {
     return T_ANY;
   }
-  if (_method & HTTP_GET) {
+  if (isMethod(HTTP_GET)) {
     return T_GET;
   }
-  if (_method & HTTP_POST) {
+  if (isMethod(HTTP_POST)) {
     return T_POST;
   }
-  if (_method & HTTP_DELETE) {
+  if (isMethod(HTTP_DELETE)) {
     return T_DELETE;
   }
-  if (_method & HTTP_PUT) {
+  if (isMethod(HTTP_PUT)) {
     return T_PUT;
   }
-  if (_method & HTTP_PATCH) {
+  if (isMethod(HTTP_PATCH)) {
     return T_PATCH;
   }
-  if (_method & HTTP_HEAD) {
+  if (isMethod(HTTP_HEAD)) {
     return T_HEAD;
   }
-  if (_method & HTTP_OPTIONS) {
+  if (isMethod(HTTP_OPTIONS)) {
     return T_OPTIONS;
   }
-  if (_method & HTTP_PROPFIND) {
+  if (isMethod(HTTP_PROPFIND)) {
     return T_PROPFIND;
   }
-  if (_method & HTTP_LOCK) {
+  if (isMethod(HTTP_LOCK)) {
     return T_LOCK;
   }
-  if (_method & HTTP_UNLOCK) {
+  if (isMethod(HTTP_UNLOCK)) {
     return T_UNLOCK;
   }
-  if (_method & HTTP_PROPPATCH) {
+  if (isMethod(HTTP_PROPPATCH)) {
     return T_PROPPATCH;
   }
-  if (_method & HTTP_MKCOL) {
+  if (isMethod(HTTP_MKCOL)) {
     return T_MKCOL;
   }
-  if (_method & HTTP_MOVE) {
+  if (isMethod(HTTP_MOVE)) {
     return T_MOVE;
   }
-  if (_method & HTTP_COPY) {
+  if (isMethod(HTTP_COPY)) {
     return T_COPY;
   }
   return T_UNKNOWN;