Skip to content

Keep splash ending from input url#52

Merged
asyrjasalo merged 1 commit intoasyrjasalo:masterfrom
vinhntb:master
Dec 8, 2018
Merged

Keep splash ending from input url#52
asyrjasalo merged 1 commit intoasyrjasalo:masterfrom
vinhntb:master

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

@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