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(docat): Trailing Slash resulting in 404 status code #269

Merged
merged 1 commit into from
Sep 19, 2022
Merged

Fix(docat): Trailing Slash resulting in 404 status code #269

merged 1 commit into from
Sep 19, 2022

Conversation

reglim
Copy link
Contributor

@reglim reglim commented Sep 19, 2022

If a slash is trailing on api functions with parameters, the api responds with 404. To fix this, I used the simple way and added a route with a trailing slash to all methods that break without it.

fixes #100

If a slash is trailing on api functions with parameters,
the api responds with 404. To fix this, I used the simple way
and added a route with a trailing slash to all methods
that break without it.

fixes #100
Copy link
Member

@randombenj randombenj left a comment

Choose a reason for hiding this comment

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

LGTM

@randombenj randombenj merged commit 36904b9 into docat-org:main Sep 19, 2022
@reglim reglim deleted the bugfix/100-trailing-slash-results-in-404 branch September 20, 2022 09:51
@fliiiix
Copy link
Member

fliiiix commented Sep 21, 2022

@reglim does this fix this issue here as well? docat-org/docatl#8

@reglim
Copy link
Contributor Author

reglim commented Sep 21, 2022

@fliiiix No, the issue https://github.com/docat-org/docatl/issues/ gives you a URL like:
http://localhost:8080//api/project
I fixed this seperately here: docat-org/docatl#9
(I just remove the trailing slash if it's there)

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.

strip extra slash on versions
3 participants