swagger-brake not reporting violation in breaking changes | MIN/MAX validation | POST object mandatory field validation about swagger-brake HOT 6 CLOSED

@madhur2928 thanks for reporting. Certainly the min/max validation is something missing from the current ruleset, however the second use-case should fail. Anyway, I'll look into both of the cases and get back to you soon.

from swagger-brake.

Fixed on master, new version will be released soon.

from swagger-brake.

thanks @galovics for fixing the issue. I took a checkout from master and tried replicating the issue.
Both min/max validation and post object mandatory field validation is working fine now.

But still facing couple of issues.

1. min/max validation is working only for GET params but not for POST fields
2. regex validation (pattern) is not working for both GET and POST

attached the json files
swagger.zip

from swagger-brake.

@madhur2928 let me take another look then at request body validations. Perhaps I missed it. :)

from swagger-brake.

@madhur2928 in fact request body validation was completely missing, so I'm doing the implementation.

Also, please bear in mind that I intentionally did not implement the regex constraint validation. In my opinion it's way too complicated to implement something like that. The only thing swagger-brake could easily verify if the pattern has changed, but IMHO that's going to result in many false positives. Somehow it should rather detect if the new regex is equal in terms of semantics or an expansion of the original regex. If you have any ideas, feel free to share.

from swagger-brake.

@madhur2928 give it a try from latest master, let's see how it goes.

from swagger-brake.