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

Keep splash ending from input url #52

Merged
merged 1 commit into from
Dec 8, 2018
Merged

Conversation

vinhntb
Copy link
Contributor

@vinhntb vinhntb commented Dec 6, 2018

Some in case API will be different with splash ending like that /api/foo/ and /api/foo. So that we should be keep original url from input.

@asyrjasalo
Copy link
Owner

Yes, this is possible. Merging, crediting and releasing next RC to PyPI straight away.

After some Googling, there is no single correct answer to this. Anyway, cannot think what this would break. There is a possibility that separate schemas will be generated, if both are used interchangeably in tests. Quite an easy fix to a seemingly rare case.

@asyrjasalo asyrjasalo merged commit e77cf69 into asyrjasalo:master Dec 8, 2018
asyrjasalo added a commit that referenced this pull request Dec 8, 2018
@vinhntb
Copy link
Contributor Author

vinhntb commented Dec 8, 2018

In my case that is Django project, I apply this library to testing API.
https://docs.djangoproject.com/en/1.11/ref/settings/#std:setting-APPEND_SLASH

@asyrjasalo
Copy link
Owner

Released: https://pypi.org/project/RESTinstance.

Thank You!

@SergeySerj
Copy link

+1 to this fix.
We also have Django project and trailing slash is mandatory for some URLs.

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