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

Getting 404 Not found when navigating to user #794

Closed
bollerdominik opened this issue May 18, 2020 · 4 comments · Fixed by #795
Closed

Getting 404 Not found when navigating to user #794

bollerdominik opened this issue May 18, 2020 · 4 comments · Fixed by #795
Labels
bug Something isn't working right

Comments

@bollerdominik
Copy link

bollerdominik commented May 18, 2020

Describe the bug
I identify my users using their email. On the All Events page I click on a person with the name t@t.com and it sends me to https://site.com/person/t%40t.com. On this page I get a 404 Page not found.

I expect to see the persons page with all their events.

Additional context
This worked previously before 1.5.0. After updating to 1.5.0 all my identified persons link are broken. Unidentified users which have the generated UUID as the distinct_Id still work

@bollerdominik bollerdominik added the bug Something isn't working right label May 18, 2020
@timgl
Copy link
Collaborator

timgl commented May 18, 2020

@mariusandra I think this is probably due to us moving to kea-router. The following url doesn't get recognised as a valid path: http://127.0.0.1:8000/person/t%40t.com. I had a quick look at the kea-router source code but couldn't figure out what was going on there.

@mariusandra
Copy link
Collaborator

@timgl indeed! PR #795 fixes this

@timgl
Copy link
Collaborator

timgl commented May 18, 2020

@bollerdominik That's fixed, thanks for raising this issue!

@bollerdominik
Copy link
Author

Amazing! Thanks for the quick fix and the great product

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working right
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants