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

Routing doesn't work when 'I' character used #1179

Closed
daltinkurt opened this issue Oct 18, 2021 · 3 comments · Fixed by #1184
Closed

Routing doesn't work when 'I' character used #1179

daltinkurt opened this issue Oct 18, 2021 · 3 comments · Fixed by #1184
Assignees
Milestone

Comments

@daltinkurt
Copy link
Contributor

I have multilanguage website.
When language is Turkish all pages work normally.
But if i choose English, the pages which contains 'I' character in url don't works (404 error).

Maybe the problem is that you use "ToLower" instead of "ToLowerInvariant" in the source code.
(or "ToUpper" instead of "ToUpperInvariant")

In Turkish:
cookie is 'c=tr|uic=tr'
image
image
...
works normally

In English:
cookie is 'c=en|uic=en'
image
image

@daltinkurt daltinkurt changed the title Routing doesn't work when used 'I' character used Routing doesn't work when 'I' character used Oct 18, 2021
@acizmarik
Copy link
Member

What DotVVM version do you use? We thought we already fixed these issues in: #1054 which is already present in version 3.2. If it still persists we need to investigate further.

@daltinkurt
Copy link
Contributor Author

daltinkurt commented Oct 19, 2021

What DotVVM version do you use? We thought we already fixed these issues in: #1054 which is already present in version 3.2. If it still persists we need to investigate further.

Dotvvm.AspNetCore 3.1.1
and
Dotvvm for Visual Studio 2019 3.1.514.0

Upgraded to 3.2.1, still doesn't work :(
image

image

@acizmarik
Copy link
Member

Thanks for report, I will look into this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants