Skip to content

OpenAPI invalid securityScheme with implicit flow (fix #4079)#4115

Merged
soyuka merged 6 commits into
api-platform:2.6from
soyuka:fix/4079
Mar 15, 2021
Merged

OpenAPI invalid securityScheme with implicit flow (fix #4079)#4115
soyuka merged 6 commits into
api-platform:2.6from
soyuka:fix/4079

Conversation

@soyuka

@soyuka soyuka commented Mar 5, 2021

Copy link
Copy Markdown
Member
Q A
Branch? 2.6
Bug fix? yes
New feature? no
Deprecations? no
Tickets Fix #4079
License MIT
Doc PR done
This has to change default configurated values.

->scalarNode('tokenUrl')->defaultValue('/oauth/v2/token')->info('The oauth token url.')->end()
->scalarNode('authorizationUrl')->defaultValue('/oauth/v2/auth')->info('The oauth authentication url.')->end()
->scalarNode('refreshUrl')->defaultValue('/oauth/v2/refresh')->info('The oauth refresh url.')->end()
->scalarNode('tokenUrl')->defaultValue('')->info('The oauth token url.')->end()

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't this a BC break?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know as it's wrong like this.

Comment thread src/OpenApi/Options.php Outdated
@soyuka soyuka merged commit f193281 into api-platform:2.6 Mar 15, 2021
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.

2 participants