René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"[Bug]: Can’t validate numerical string as `type`: `integer`/`number`/`boolean` wrapped in `allOf`/`anyOf`/`oneOf`","articleBody":"### Actual Behavior\n\nThe string `123` from a path or query parameter validates as `{\"type\": \"integer\"}`, but unexpectedly fails to validate as `{\"allOf\": [{\"type\": \"integer\"}]}`. The same problem occurs with `number` or `boolean` in place of `integer`, or `anyOf` or `oneOf` in place of `allOf`.\r\n\r\n(My [actual use case](https://github.com/zulip/zulip/blob/1839eb023977107699e2368ccdb0f20a496f6fcf/zerver/openapi/zulip.yaml#L17561-L17568) involves `{\"oneOf\": [{\"type\": \"string\", \"enum\": [\"newest\", \"oldest\", \"first_unread\"]}, {\"type\": \"integer\"}]}`.)\r\n\r\n```pytb\r\nTraceback (most recent call last):\r\n  File \"/home/anders/python/openapi-core/openapi_core/validation/decorators.py\", line 31, in wrapper\r\n    return f(*args, **kwds)\r\n  File \"/home/anders/python/openapi-core/openapi_core/validation/request/validators.py\", line 200, in _get_parameter\r\n    value, _ = self._get_param_or_header_and_schema(param, location)\r\n  File \"/home/anders/python/openapi-core/openapi_core/validation/validators.py\", line 166, in _get_param_or_header_and_schema\r\n    self._validate_schema(schema, casted)\r\n  File \"/home/anders/python/openapi-core/openapi_core/validation/validators.py\", line 144, in _validate_schema\r\n    validator.validate(value)\r\n  File \"/home/anders/python/openapi-core/openapi_core/validation/schemas/validators.py\", line 36, in validate\r\n    raise InvalidSchemaValue(value, schema_type, schema_errors=errors)\r\nopenapi_core.validation.schemas.exceptions.InvalidSchemaValue: Value 123 not valid for schema of type any: (\u003cValidationError: \"'123' is not of type 'integer'\"\u003e,)\r\n\r\nThe above exception was the direct cause of the following exception:\r\n\r\nTraceback (most recent call last):\r\n  File \"/home/anders/python/openapi-core/test.py\", line 41, in \u003cmodule\u003e\r\n    validate_request(request, spec=spec)  # error\r\n  File \"/home/anders/python/openapi-core/openapi_core/shortcuts.py\", line 321, in validate_request\r\n    validate_apicall_request(\r\n  File \"/home/anders/python/openapi-core/openapi_core/shortcuts.py\", line 396, in validate_apicall_request\r\n    return v.validate(request)\r\n  File \"/home/anders/python/openapi-core/openapi_core/validation/request/validators.py\", line 279, in validate\r\n    raise err\r\n  File \"/home/anders/python/openapi-core/openapi_core/validation/request/validators.py\", line 164, in _get_parameters\r\n    value = self._get_parameter(parameters, param)\r\n  File \"/home/anders/python/openapi-core/openapi_core/validation/decorators.py\", line 33, in wrapper\r\n    self._raise_error(exc, self.err_validate_cls, f, *args, **kwds)\r\n  File \"/home/anders/python/openapi-core/openapi_core/validation/decorators.py\", line 58, in _raise_error\r\n    raise init(**kw) from exc\r\nopenapi_core.validation.request.exceptions.InvalidParameter: Invalid path parameter: bar_id\r\n```\n\n### Expected Behavior\n\nNo error. A string that validates as `{\"type\": \"integer\"}` should also validate as `{\"allOf\": [{\"type\": \"integer\"}]}`.\n\n### Steps to Reproduce\n\n```python\r\nfrom openapi_core import Spec, validate_request\r\nfrom openapi_core.testing import MockRequest\r\n\r\nspec = Spec.from_dict(\r\n    {\r\n        \"openapi\": \"3.1.0\",\r\n        \"info\": {\"title\": \"test\", \"version\": \"0\"},\r\n        \"paths\": {\r\n            \"/foo/{foo_id}\": {\r\n                \"get\": {\r\n                    \"parameters\": [\r\n                        {\r\n                            \"name\": \"foo_id\",\r\n                            \"in\": \"path\",\r\n                            \"required\": True,\r\n                            \"schema\": {\"type\": \"integer\"},\r\n                        },\r\n                    ],\r\n                },\r\n            },\r\n            \"/bar/{bar_id}\": {\r\n                \"get\": {\r\n                    \"parameters\": [\r\n                        {\r\n                            \"name\": \"bar_id\",\r\n                            \"in\": \"path\",\r\n                            \"required\": True,\r\n                            \"schema\": {\"allOf\": [{\"type\": \"integer\"}]},\r\n                        },\r\n                    ],\r\n                },\r\n            },\r\n        },\r\n    }\r\n)\r\n\r\nrequest = MockRequest(\"http://localhost\", \"get\", \"/foo/123\")\r\nvalidate_request(request, spec=spec)  # ok\r\n\r\nrequest = MockRequest(\"http://localhost\", \"get\", \"/bar/123\")\r\nvalidate_request(request, spec=spec)  # error\r\n```\n\n### OpenAPI Core Version\n\n0.18.1 or current Git (0838a84e7c23d9f0691e0e7d496550d12513476e)\n\n### OpenAPI Core Integration\n\nnone\n\n### Affected Area(s)\n\n_No response_\n\n### References\n\n_No response_\n\n### Anything else we need to know?\n\n_No response_\n\n### Would you like to implement a fix?\n\nNone","author":{"url":"https://github.com/andersk","@type":"Person","name":"andersk"},"datePublished":"2023-10-21T00:10:31.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":4},"url":"https://github.com/698/openapi-core/issues/698"}