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

fix(doc) : Add slash to the documentation for fix swagger #1883

Open
wants to merge 1 commit into
base: 3.2
Choose a base branch
from

Conversation

SalvadorCardona
Copy link

@SalvadorCardona SalvadorCardona commented Jan 19, 2024

If the / of auth is missing in Api Swager the Url should be localhostauth

If you add / the url should localhost/auth

Before :

image

After :

image

If the / of auth is missing in Api Swager the Url should be localhostauth

If you add / the url should localhost/auth
@SalvadorCardona SalvadorCardona changed the title Update jwt.md fix(doc) : Add slash to the documentation for fix swagger Jan 19, 2024
@@ -85,7 +85,7 @@ security:
stateless: true
provider: users
json_login:
check_path: auth # The name in routes.yaml is enough for mapping
check_path: /auth # The name in routes.yaml is enough for mapping
Copy link
Member

Choose a reason for hiding this comment

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

imo this is a name of a route declared inside symfony routing no?

Copy link
Author

Choose a reason for hiding this comment

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

Maybe it's the problem that I updated my symfony to version 7 but the name of my route is auth

image

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