Replace validate_definition with validate_ref

[kehitt]

Кажется, что спецификация OAS2 протекла в валидатор JSON Schema. Сейчас это мешает переехать на OAS3, пути то поменялись.

[keynslug]

Сейчас это мешает переехать на OAS3, пути то поменялись.

Как интересно. А можешь ссылок покидать на те куски OAS3, которые это затронуло.

Я просто не понимаю пока, почему мы рассуждаем о протечке. Я вижу, что definitions – это вроде как часть спецификации json schema, в том числе draft 05:

• https://tools.ietf.org/html/draft-fge-json-schema-validation-00#section-5.5.7
• https://tools.ietf.org/html/draft-wright-json-schema-validation-00#section-5.26

[keynslug]

Просто чтобы было больше контекста по поводу того, зачем там validate_definition вообще появился:

for-GET#53 (comment)

[kehitt]

@keynslug
Странно, я смотрел тут, допустим, и там уже используется $defs. Потому у меня сложилось впечатление, что это скорее рекомендация, чем часть стандарта. И ещё потому, что апстрим тоже избегает подобного хардкода путей.
https://json-schema.org/draft/2019-09/json-schema-core.html#rfc.section.4

В OAS3, допустим, все определения разъехались по различным вариациям пути #/components/{componentType} https://github.com/OAI/OpenAPI-Specification/blob/master/versions/3.0.3.md#componentsObject

Контекст мне понятен, так-то, ничего против не имею, просто у OAS3-схемы в #/definitionsничего нет.

[keynslug]

Странно, я смотрел тут, допустим, и там уже используется $defs. Потому у меня сложилось впечатление, что это скорее рекомендация, чем часть стандарта. И ещё потому, что апстрим тоже избегает подобного хардкода путей.
https://json-schema.org/draft/2019-09/json-schema-core.html#rfc.section.4

Судя по https://github.com/OAI/OpenAPI-Specification/blob/master/versions/3.0.3.md#schemaObject OAS3 пока всё ещё на древнем draft 04/05. Поэтому на него ссылки и скинул. Иначе нам бы пришлось jesse допиливать до поддержки draft-2019-09. Быть может что-то не доглядел конечно.

Контекст мне понятен, так-то, ничего против не имею, просто у OAS3-схемы в #/definitionsничего нет.

Понял. Тогда быть может ввести validate_ref (validate_local_ref?) с такими же гарантиями в дополнение к validate_definition? После чего validate_definition поверх этой новой функции выразить. Не вижу просто смысла от него избавляться сейчас.

[kehitt]

Тогда быть может ввести validate_ref (validate_local_ref?) с такими же гарантиями в дополнение к validate_definition?

Да, теперь думаю что так будет лучше, раз статус definitions не так однозначен, как я думал.