From c34f8cbccd44a88a302999a19f70cfd3bff5cbef Mon Sep 17 00:00:00 2001 From: Joost van der Hoff Date: Fri, 3 May 2019 12:37:02 +0200 Subject: [PATCH 1/4] Add option to include request and response body schemas Include schemas from: paths///requestBody/content//schema paths///responses//content//schema Closes #25 --- openapi2jsonschema/command.py | 45 ++++++++++++++++++++++++++++++++++- openapi2jsonschema/util.py | 16 +++++++++++++ 2 files changed, 60 insertions(+), 1 deletion(-) diff --git a/openapi2jsonschema/command.py b/openapi2jsonschema/command.py index 8261f2a..0e721cb 100644 --- a/openapi2jsonschema/command.py +++ b/openapi2jsonschema/command.py @@ -5,6 +5,7 @@ import urllib import os import sys +import re from jsonref import JsonRef # type: ignore import click @@ -16,6 +17,7 @@ allow_null_optional_fields, change_dict_values, append_no_duplicates, + get_components_from_body_definition, ) from openapi2jsonschema.errors import UnsupportedError @@ -48,8 +50,22 @@ is_flag=True, help="Prohibits properties not in the schema (additionalProperties: false)", ) +@click.option( + "--include-bodies", + is_flag=True, + help="Include request and response bodies as if they are components", +) @click.argument("schema", metavar="SCHEMA_URL") -def default(output, schema, prefix, stand_alone, expanded, kubernetes, strict): +def default( + output, + schema, + prefix, + stand_alone, + expanded, + kubernetes, + strict, + include_bodies, +): """ Converts a valid OpenAPI specification into a set of JSON Schema files """ @@ -120,6 +136,33 @@ def default(output, schema, prefix, stand_alone, expanded, kubernetes, strict): else: components = data["components"]["schemas"] + if include_bodies: + for path, path_definition in data["paths"].items(): + for http_method, http_method_definition in path_definition.items(): + name_prefix_fmt = "paths_{:s}_{:s}_{{:s}}_".format( + # Paths "/" and "/root" will conflict, + # no idea how to solve this elegantly. + path.lstrip("/").replace("/", "_") or "root", + http_method.upper(), + ) + name_prefix_fmt = re.sub( + r"\{([^:\}]+)\}", + r"_\1_", + name_prefix_fmt, + ) + if "requestBody" in http_method_definition: + components.update(get_components_from_body_definition( + http_method_definition["requestBody"], + prefix=name_prefix_fmt.format("request") + )) + responses = http_method_definition["responses"] + for response_code, response in responses.items(): + response_name_part = "response_{}".format(response_code) + components.update(get_components_from_body_definition( + response, + prefix=name_prefix_fmt.format(response_name_part), + )) + for title in components: kind = title.split(".")[-1].lower() if kubernetes: diff --git a/openapi2jsonschema/util.py b/openapi2jsonschema/util.py index 68fd8d4..9370bbd 100644 --- a/openapi2jsonschema/util.py +++ b/openapi2jsonschema/util.py @@ -111,3 +111,19 @@ def append_no_duplicates(obj, key, value): obj[key] = [] if value not in obj[key]: obj[key].append(value) + + +def get_components_from_body_definition(body_definition, prefix=""): + MIMETYPE_TO_TYPENAME_MAP = { + "application/json": "json", + "application/vnd.api+json": "jsonapi", + } + result = {} + for mimetype, definition in body_definition["content"].items(): + type_name = MIMETYPE_TO_TYPENAME_MAP.get( + mimetype, + mimetype.replace("/", "_"), + ) + if "schema" in definition: + result["{:s}{:s}".format(prefix, type_name)] = definition["schema"] + return result From 11d254b177eff1272fff8e9941b57184e4c8a094 Mon Sep 17 00:00:00 2001 From: Joost van der Hoff Date: Fri, 3 May 2019 12:46:03 +0200 Subject: [PATCH 2/4] Do not downcase filenames Closes #20 --- openapi2jsonschema/command.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/openapi2jsonschema/command.py b/openapi2jsonschema/command.py index 0e721cb..a022738 100644 --- a/openapi2jsonschema/command.py +++ b/openapi2jsonschema/command.py @@ -164,7 +164,7 @@ def default( )) for title in components: - kind = title.split(".")[-1].lower() + kind = title.split(".")[-1] if kubernetes: group = title.split(".")[-3].lower() api_version = title.split(".")[-2].lower() @@ -198,8 +198,7 @@ def default( if ( kubernetes and stand_alone - and kind - in [ + and kind.lower() in [ "jsonschemaprops", "jsonschemapropsorarray", "customresourcevalidation", From 483eda6f2712639c2bc0338eefb942d9f1d57683 Mon Sep 17 00:00:00 2001 From: Joost van der Hoff Date: Fri, 3 May 2019 12:53:58 +0200 Subject: [PATCH 3/4] Move new functionality to separate function --- openapi2jsonschema/command.py | 31 ++++--------------------------- openapi2jsonschema/util.py | 32 ++++++++++++++++++++++++++++++++ 2 files changed, 36 insertions(+), 27 deletions(-) diff --git a/openapi2jsonschema/command.py b/openapi2jsonschema/command.py index a022738..305235b 100644 --- a/openapi2jsonschema/command.py +++ b/openapi2jsonschema/command.py @@ -5,7 +5,6 @@ import urllib import os import sys -import re from jsonref import JsonRef # type: ignore import click @@ -17,7 +16,7 @@ allow_null_optional_fields, change_dict_values, append_no_duplicates, - get_components_from_body_definition, + get_request_and_response_body_components_from_paths, ) from openapi2jsonschema.errors import UnsupportedError @@ -137,31 +136,9 @@ def default( components = data["components"]["schemas"] if include_bodies: - for path, path_definition in data["paths"].items(): - for http_method, http_method_definition in path_definition.items(): - name_prefix_fmt = "paths_{:s}_{:s}_{{:s}}_".format( - # Paths "/" and "/root" will conflict, - # no idea how to solve this elegantly. - path.lstrip("/").replace("/", "_") or "root", - http_method.upper(), - ) - name_prefix_fmt = re.sub( - r"\{([^:\}]+)\}", - r"_\1_", - name_prefix_fmt, - ) - if "requestBody" in http_method_definition: - components.update(get_components_from_body_definition( - http_method_definition["requestBody"], - prefix=name_prefix_fmt.format("request") - )) - responses = http_method_definition["responses"] - for response_code, response in responses.items(): - response_name_part = "response_{}".format(response_code) - components.update(get_components_from_body_definition( - response, - prefix=name_prefix_fmt.format(response_name_part), - )) + components.update( + get_request_and_response_body_components_from_paths(data["paths"]), + ) for title in components: kind = title.split(".")[-1] diff --git a/openapi2jsonschema/util.py b/openapi2jsonschema/util.py index 9370bbd..8424e51 100644 --- a/openapi2jsonschema/util.py +++ b/openapi2jsonschema/util.py @@ -1,5 +1,7 @@ #!/usr/bin/env python +import re + def iteritems(d): if hasattr(dict, "iteritems"): @@ -127,3 +129,33 @@ def get_components_from_body_definition(body_definition, prefix=""): if "schema" in definition: result["{:s}{:s}".format(prefix, type_name)] = definition["schema"] return result + + +def get_request_and_response_body_components_from_paths(paths): + components = {} + for path, path_definition in paths.items(): + for http_method, http_method_definition in path_definition.items(): + name_prefix_fmt = "paths_{:s}_{:s}_{{:s}}_".format( + # Paths "/" and "/root" will conflict, + # no idea how to solve this elegantly. + path.lstrip("/").replace("/", "_") or "root", + http_method.upper(), + ) + name_prefix_fmt = re.sub( + r"\{([^:\}]+)\}", + r"_\1_", + name_prefix_fmt, + ) + if "requestBody" in http_method_definition: + components.update(get_components_from_body_definition( + http_method_definition["requestBody"], + prefix=name_prefix_fmt.format("request") + )) + responses = http_method_definition["responses"] + for response_code, response in responses.items(): + response_name_part = "response_{}".format(response_code) + components.update(get_components_from_body_definition( + response, + prefix=name_prefix_fmt.format(response_name_part), + )) + return components From 72d89f140a2bef8b70df0febe90380ed041c7fa9 Mon Sep 17 00:00:00 2001 From: Zeust the Unoobian Date: Thu, 15 Oct 2020 10:19:35 +0200 Subject: [PATCH 4/4] Make "content" key in body definition optional This was committed as a GitHub review suggestion. Commit message suggested by Github: Update openapi2jsonschema/util.py Co-authored-by: Adam Kitain --- openapi2jsonschema/util.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/openapi2jsonschema/util.py b/openapi2jsonschema/util.py index 8424e51..2e881c7 100644 --- a/openapi2jsonschema/util.py +++ b/openapi2jsonschema/util.py @@ -121,7 +121,7 @@ def get_components_from_body_definition(body_definition, prefix=""): "application/vnd.api+json": "jsonapi", } result = {} - for mimetype, definition in body_definition["content"].items(): + for mimetype, definition in body_definition.get("content", {}).items(): type_name = MIMETYPE_TO_TYPENAME_MAP.get( mimetype, mimetype.replace("/", "_"),