feat: api docs remove query string from get by barcode and add security to solve openapi linter errors/warnings #10922
+1,545
−56
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What
This request was born from the desire to fix some validation errors raised by the most common SDK client generators (openapi, swagger, redocly). The pull request unifies the 2 definitions of
get by barcode
and addssecurity schemes
and authentication endpoints to the Open Food Facts API.cookieAuth
anduserAgentAuth
./cgi/session.pl
for login and obtaining a session cookie.AppCredentials
.Authentication
for endpoints related to authentication.get by barcode
and removed path with query string to solve Redocly and Swagger validation error: "Problem: Don't put query string items in the path, they belong in parameters within: query
."Screenshot
Related issue(s) and discussion
📝 API Documentation using OpenAPI (tracker) #7446
Usability issues with the API doc #9419
OpenAPISpec is not a valid 3.1.0 openAPI-Spec #8039