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

Something leaks '/undefined' into URLs when acting as a guest #1079

Closed
ara4n opened this issue Mar 3, 2016 · 8 comments
Closed

Something leaks '/undefined' into URLs when acting as a guest #1079

ara4n opened this issue Mar 3, 2016 · 8 comments
Labels
P2 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect

Comments

@ara4n
Copy link
Member

ara4n commented Mar 3, 2016

I just started a fresh incognito tab against my local develop vector. This put me as a guest in RoomDirectory view, but something rewrote the location bar as http://localhost:8080/#/room/undefined. I then hit refresh, which took me to a 'Join' page, presumably trying to join a room with room id of "undefined". This also breaks URL-keyed page stats :(

@ara4n ara4n added T-Defect P2 S-Major Severely degrades major functionality or product features, with no satisfactory workaround labels Mar 3, 2016
@ara4n ara4n added this to the v1 - Private Preview milestone Mar 3, 2016
@ara4n ara4n added S-Minor Impairs non-critical functionality or suitable workarounds exist and removed S-Major Severely degrades major functionality or product features, with no satisfactory workaround labels Mar 3, 2016
@mebjas
Copy link

mebjas commented Mar 4, 2016

What is the expected behavior here?
@ara4n

@ara4n
Copy link
Member Author

ara4n commented Mar 4, 2016

The app should take you to RoomDirectory view on initial launch, but the URL should either be something explicit (like /directory) and certainly not /#/room/undefined.

@mebjas
Copy link

mebjas commented Mar 5, 2016

@ara4n now shows /directory

@ara4n
Copy link
Member Author

ara4n commented Mar 15, 2016

I'm still seeing this happen - e.g. loading http://localhost:8080/#/login leaves me sitting at http://localhost:8080/#/room/undefined

@mebjas
Copy link

mebjas commented Mar 15, 2016

Lemme check

@mebjas
Copy link

mebjas commented Mar 15, 2016

@ara4n This is strange, loading http://localhost:8080/#/login took me to http://localhost:8080/#/directory

More pointers on steps to reproduce?

@ara4n
Copy link
Member Author

ara4n commented Mar 15, 2016

gah, i think i was on an old feature branch - sorry :( on the plus side,
yay! i'll close this

On 15/03/2016 16:00, minhaz wrote:

@ara4n https://github.com/ara4n This is strange, loading
|http://localhost:8080/#/login| took me to
|http://localhost:8080/#/directory|

More pointers on steps to reproduce?


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
#1079 (comment)

@ara4n
Copy link
Member Author

ara4n commented Mar 15, 2016

fixed - thanks @mebjas :)

@ara4n ara4n closed this as completed Mar 15, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect
Projects
None yet
Development

No branches or pull requests

2 participants