krakenjs / swaggerize-routes

Swagger document driven route builder.
Other
58 stars 57 forks source link

bugfix(validator): switch from string to array of string #55

Closed kenjones-cisco closed 8 years ago

kenjones-cisco commented 8 years ago

Update the consumes of file type to be array with items being a string with pattern.

closes #54

tlivings commented 8 years ago

Is there a situation where you'd have more than one content-type header?

kenjones-cisco commented 8 years ago

No, but to align to the swagger spec, so that writing the API definition following the swagger spec expects the definition to be a list. It causes an error in Swagger Editor.

tlivings commented 8 years ago

Cool. I think reviewing this has actually revealed a gaping hole in the validation of file type :(

subeeshcbabu-zz commented 8 years ago

lgtm :+1: