Fix case when schema doesn't have defined required field.

PayU / openapi-validator-middleware

Input validation using Swagger (Open API) and ajv

Apache License 2.0

144 stars 50 forks source link

Fix case when schema doesn't have defined required field. #24

Closed kibertoad closed 6 years ago

kibertoad commented 6 years ago

refs #21

kibertoad commented 6 years ago

@idanto Aaaand the very first attempt to use new code in production uncovered a new way this can crash :).

coveralls commented 6 years ago

Pull Request Test Coverage Report for Build 94

1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.004%) to 99.333%

Totals
Change from base Build 93:	0.004%
Covered Lines:	184
Relevant Lines:	184

💛 - Coveralls

kibertoad commented 6 years ago

@idanto Has the node security check died?..

idanto commented 6 years ago

not sure what happen there maybe it because of the project transfer, I will check that later. Either way cause it's a bug and you didn't add or change any dependency all good and merged