Open ikokostya opened 5 years ago
This is a good suggestion for a validation rule. Thanks!
Just to be clear, z-schema
(JSON schema validation library) doesn't check this too:
const ZSchema = require('z-schema');
const validator = new ZSchema();
const schema = {
type: 'object',
properties: {
foo: {
type: 'string'
}
},
additionalProperties: false,
required: ['bar']
};
console.log(validator.validateSchema(schema)); // true
Just to understanda the issue... why the schema in the OP is supposed to be wrong?? In my view, the schema is correct, and it simply says that property "bar" is required, and must always be present in the response, but the schema does not constrain what type it can take. "bar" can be an integer, a string, an object... whatever.
Right. It's more of a lint rule than a schema violation
Isn't this kind of error a sign that the specification is incorrect. Trying to understand how can a property can be "required" but not defined in the schema. The swagger 2.0 validation option should treat this case as an error, right ? (validate options - validate.spec:true
).
The swagger 2.0 validation behaviour is that a validation error as per OP was raised and I think it should be also the behaviour to move forward.
Late edit: Functionality is present in the last version, apollogies for confusion i thought it was removed.
Fixed in https://github.com/APIDevTools/swagger-parser/pull/179 - All the spec consistency checks were only used to work on Swagger 2.0 specs; All fixed now for OpenAPI v3.0 specs; Might get a chance to look into v3.1 changes later.
Versions
8.12.0
6.0.2
Code example
API specification:
Expected behavior
Validation error: property
bar
is required, but it's not defined.Actual behavior
No errors.