Skip to content

Fix Configuration Bug for Api Keys#1351

Merged
Simperfit merged 2 commits into
api-platform:masterfrom
Engerim:patch-1
Sep 6, 2017
Merged

Fix Configuration Bug for Api Keys#1351
Simperfit merged 2 commits into
api-platform:masterfrom
Engerim:patch-1

Conversation

@Engerim

@Engerim Engerim commented Sep 6, 2017

Copy link
Copy Markdown
Contributor
Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #1240
License MIT

This fix a configuration bug which was added with #1240.

The Configuration was implement in a way that the config for api keys looks like this.

swagger:
    api_keys:
      name: "key"
      type: "query"

But the implementation expects an array for api keys so the config should look like

swagger:
    api_keys:
      -
        name: "key"
        type: "query"

@Simperfit

Copy link
Copy Markdown
Contributor

Thanks @Engerim

@Simperfit Simperfit merged commit da269e8 into api-platform:master Sep 6, 2017
hoangnd25 pushed a commit to hoangnd25/core that referenced this pull request Feb 23, 2018
Fix Configuration Bug for Api Keys
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants