api: Don't restrict unknown querystring parameters yet (redux)

We missed a few in change I1e862cb1c5e9c218cea59800ff759a1b094b5906. We
also missed a few comments in places that we can change later.

Change-Id: I6fc40baf536605d9d347741bcf035958b8490b07
Signed-off-by: Stephen Finucane <stephenfin@redhat.com>

keystone/catalog/schema.py

@@ -98,6 +98,8 @@ service_schema: dict[str, Any] = {
 service_index_request_query: dict[str, Any] = {
     "type": "object",
     "properties": {},
+    # TODO(stephenfin): Change this to False once we have schemas for all
+    # resources.
     "additionalProperties": True,
 }
 
@@ -278,7 +280,9 @@ endpoint_index_response_body: dict[str, Any] = {
 endpoint_request_query: dict[str, Any] = {
     "type": "object",
     "properties": {},
-    "additionalProperties": False,
+    # TODO(stephenfin): Change this to False once we have schemas for all
+    # resources.
+    "additionalProperties": True,
 }
 
 # Response of the `/endpoints` API returning a single endpoint
@@ -406,6 +410,8 @@ endpoint_group_index_request_query: dict[str, Any] = {
             "description": "The name of the endpoint group.",
         }
     },
+    # TODO(stephenfin): Change this to False once we have schemas for all
+    # resources. Doing so will remove comparator (name__icontains) support.
     "additionalProperties": False,
 }
 

keystone/federation/schema.py

@@ -213,6 +213,8 @@ identity_provider_index_request_query: dict[str, Any] = {
             "description": "Whether the identity provider is enabled or not",
         },
     },
+    # TODO(stephenfin): Change this to False once we have schemas for all
+    # resources. Doing so will remove comparator (name__icontains) support.
     "additionalProperties": True,
 }
 
@@ -232,6 +234,8 @@ identity_provider_index_response_body: dict[str, Any] = {
 identity_provider_request_query: dict[str, Any] = {
     "type": "object",
     "properties": {},
+    # TODO(stephenfin): Change this to False once we have schemas for all
+    # resources.
     "additionalProperties": True,
 }
 

keystone/identity/schema.py

@@ -208,7 +208,9 @@ group_index_request_query: dict[str, Any] = {
         "sort_key": parameter_types.sort_key,
         "sort_dir": parameter_types.sort_dir,
     },
-    "additionalProperties": False,
+    # TODO(stephenfin): Change this to False once we have schemas for all
+    # resources. Doing so will remove comparator (name__icontains) support.
+    "additionalProperties": True,
 }
 
 _group_properties: dict[str, Any] = {