search

EXSyst / ApiDocBundle

UNMAINTAINED - Use NelmioApiDocBundle instead
MIT License
1 stars 0 forks source link

Conflict between symfony path parameter and swagger specification #3

Closed magnetik closed 7 years ago

magnetik commented 7 years ago

Using FOSRestBundle, all routes have the path defined as "/foo/bar.{_format}"

The _format has a default value, but open api specification state: 

required: If the parameter is in "path", this property is required and its value MUST be true.
default: (Note: "default" has no meaning for required parameters.)
GuilhemN commented 7 years ago

The only "solution" I found is removing .{_format}. Maybe we should ask if this limitation can be removed in the next version of the specification.

magnetik commented 7 years ago

Well for me the fix does not work: even with the {._format} removed from path, the RouteMetadataDescriber gets it from the compiled route and it's added.

GuilhemN commented 7 years ago

Is https://github.com/EXSyst/ApiDocBundle/pull/5 fine for you or do you see another solution?

GuilhemN commented 7 years ago

Fixed in #5 :)