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

Fixed api calls with double slashes #32

Merged
merged 2 commits into from
Oct 13, 2021
Merged

Fixed api calls with double slashes #32

merged 2 commits into from
Oct 13, 2021

Conversation

dbeuchler
Copy link
Member

The wcf api was more graceful and allows double slashes in api urls. When hosted under kestrel, double slashes are not possible anymore. I fixed the double slashes for the api calls to only use one slash.

@Toxantron i would classify this as hot fix and will release a 3.1.1 if this was merged.

@dbeuchler dbeuchler added the bug Something isn't working label Oct 13, 2021
@dbeuchler dbeuchler added this to the MaintenanceWeb 3.1 milestone Oct 13, 2021
@dbeuchler dbeuchler requested a review from Toxantron October 13, 2021 12:17
@dbeuchler dbeuchler self-assigned this Oct 13, 2021
@Toxantron
Copy link
Member

I agree

@Toxantron Toxantron merged commit dcbebe9 into dev Oct 13, 2021
@Toxantron Toxantron deleted the fix/apiCalls branch October 13, 2021 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants