Remove URIMatchAll to avoid any mis-use form users

Author: mathieucarbou

src/ESPAsyncWebServer.h

@@ -733,7 +733,6 @@ class AsyncWebServerRequest {
 
 enum AsyncURIMatchFlags {
   // Meta flags - low bits
-  URIMatchAll = 0,          // No flags set
   URIMatchAuto = (1 << 0),  // parse _uri at construct time and infer match type(s)
                             // (_uri may be transformed to remove wildcards)
   URIMatchCaseInsensitive = (1 << 1),
@@ -743,7 +742,6 @@ enum AsyncURIMatchFlags {
   URIMatchPrefix = (1 << 30),        // matches equivalent to regex: ^{_uri}.*
   URIMatchPrefixFolder = (1 << 29),  // matches equivalent to regex: ^{_uri}/.*
   URIMatchExtension = (1 << 28),     // non-regular match: /pattern../*.ext
-                                     // TODO - suffix match?
 
 #ifdef ASYNCWEBSERVER_REGEX
   URIMatchRegex = (1 << 27),  // matches _url as regex
@@ -752,7 +750,7 @@ enum AsyncURIMatchFlags {
 
 class AsyncURIMatcher {
 public:
-  AsyncURIMatcher() : _flags(URIMatchAll) {}
+  AsyncURIMatcher() {}
   AsyncURIMatcher(String uri, int flags = URIMatchAuto) : _value(std::move(uri)), _flags(flags) {
 #ifdef ASYNCWEBSERVER_REGEX
     if ((_flags & URIMatchRegex) || ((_flags & URIMatchAuto) && _value.startsWith("^") && _value.endsWith("$"))) {
@@ -764,12 +762,6 @@ class AsyncURIMatcher {
       _value.toLowerCase();
     }
     if (_flags & URIMatchAuto) {
-      // Inspect _value to set flags
-      // empty URI matches everything
-      if (!_value.length()) {
-        _flags = URIMatchAll;
-        return;
-      }
       // wildcard match with * at the end
       if (_value.endsWith("*")) {
         _flags |= URIMatchPrefix;
@@ -866,7 +858,7 @@ class AsyncURIMatcher {
 #endif
 
     // empty URI matches everything
-    if (_flags & URIMatchAll) {
+    if (!_value.length()) {
       return true;
     }