Skip to content

Conversation

@jonag
Copy link
Contributor

@jonag jonag commented Aug 4, 2023

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets -
License MIT
Doc PR -

With #5609 the integration of API Platform with LexikJWTAuthenticationBundle is automatically enabled. However, if we are not using the default configuration of LexikJWTAuthenticationBundle the documented operation is wrong (wrong path, wrong response in my case) and creates a duplicate. Since the behaviour of API Platform changed in a patch version maybe it's a good idea to explain directly in the changelog how to go back to the previous behaviour.

…uthenticationBundle

With api-platform#5609 the integration of API Platform with LexikJWTAuthenticationBundle is automatically enabled. However, if we are not using the default configuration of LexikJWTAuthenticationBundle the documented operation is wrong (wrong path, wrong response in my case) and creates a duplicate.
Since the behaviour of API Platform changed in a patch version maybe it's a good idea to explain directly in the changelog how to go back to the previous behaviour.
@divine
Copy link
Contributor

divine commented Aug 4, 2023

That behavior was incorrect and a "bug" that was fixed. In my case, it worked fine before (enabled automatically) after the upgrade I found it doesn't work anymore.

Maybe push your docs suggestion on how to disable it to https://github.com/api-platform/docs/blob/3.1/core/jwt.md or https://github.com/lexik/LexikJWTAuthenticationBundle/blob/2.x/Resources/doc/index.rst#configuration?

Thanks!

@soyuka
Copy link
Member

soyuka commented Aug 5, 2023

we shouldn't prepend if you configured it manually, imo https://github.com/api-platform/core/pull/5609/files needs a patch no?

@soyuka soyuka merged commit 78a4966 into api-platform:3.1 Aug 8, 2023
@soyuka
Copy link
Member

soyuka commented Aug 8, 2023

Merging this for now, don't hesitate to provide a patch if there's something better to be done. Thanks!

@jonag
Copy link
Contributor Author

jonag commented Aug 8, 2023

Detecting if a user has previously manually documented the login endpoint would be quite difficult, so I think it's good as-is.

Enabling the API Platform integration when it's not needed just adds some "noise" in the swagger documentation but doesn't break existing apps, so it should be good.

Romaixn pushed a commit to Romaixn/core that referenced this pull request Oct 3, 2023
…uthenticationBundle (api-platform#5719)

With api-platform#5609 the integration of API Platform with LexikJWTAuthenticationBundle is automatically enabled. However, if we are not using the default configuration of LexikJWTAuthenticationBundle the documented operation is wrong (wrong path, wrong response in my case) and creates a duplicate.
Since the behaviour of API Platform changed in a patch version maybe it's a good idea to explain directly in the changelog how to go back to the previous behaviour.
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.

3 participants