Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Security GET parameters are rejected by strict validation #470

Closed
underyx opened this issue Jun 19, 2017 · 0 comments
Closed

Security GET parameters are rejected by strict validation #470

underyx opened this issue Jun 19, 2017 · 0 comments
Labels

Comments

@underyx
Copy link
Contributor

underyx commented Jun 19, 2017

Description

If I have the following definition, connexion will reject all /ping requests with Extra query parameter(s) api_key not in spec.

securityDefinitions:
  ApiKeyAuth:
    type: apiKey
    in: query
    name: api_key
paths:
  /ping:
    security:
      - ApiKeyAuth: []
    get:
      summary: Return a simple JSON payload for health check purposes
      responses:
        #

Expected behaviour

connexion should know that api_key is also a supported parameter name.

Actual behaviour

connexion thinks api_key has no place to be there

Steps to reproduce

  1. Launch an app with the above Swagger file
  2. Send a request to /ping?api_key=1

Additional info:

Output of the commands:

  • python --version: Python 2.7.13
  • pip show connexion | grep "^Version\:": Version: 1.1.9
svetlana-v pushed a commit to svetlana-v/connexion that referenced this issue Nov 20, 2019
svetlana-v pushed a commit to svetlana-v/connexion that referenced this issue Nov 22, 2019
svetlana-v pushed a commit to svetlana-v/connexion that referenced this issue Nov 24, 2019
svetlana-v pushed a commit to svetlana-v/connexion that referenced this issue Nov 24, 2019
svetlana-v pushed a commit to svetlana-v/connexion that referenced this issue Nov 24, 2019
svetlana-v pushed a commit to svetlana-v/connexion that referenced this issue Dec 5, 2019
svetlana-v pushed a commit to svetlana-v/connexion that referenced this issue Dec 5, 2019
svetlana-v pushed a commit to svetlana-v/connexion that referenced this issue Dec 5, 2019
svetlana-v pushed a commit to svetlana-v/connexion that referenced this issue Dec 10, 2019
svetlana-v pushed a commit to svetlana-v/connexion that referenced this issue Dec 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants