Navbar doesn't show logout due to timeout

[barroudjo]

This deserves a bit of explanation: when you log in as admin, and let your session lose its validity (for example because you put computer in standby), the views reflect it: you can't access those that require admin auth.
However the navbar keeps the display from when you were logged in.
Attached is an example showing this situation, after clicking on admin on the navbar.

[gintsgints]

Not sure that session is "loosed it's validity" in your screen. What if you go Admin menu in that state?

[barroudjo]

That's precisely what I did, and what brought me to this screen. The admin button, among other things in the navbar, should not be available if the session is not valid and the user is not recognized as admin.

[gintsgints]

So what happens if you click to that admin menu?

[barroudjo]

Precisely what is shown in the screenshot: I get the login screen.

The problem is clear: if you're redirected to the login screen, then the navbar shouldn't still have the admin button available...

[kingcody]

@barroudjo you are absolutely correct, that is not intended behavior. Just so I'm on the same page as you, are you using the current stable (master) branch of this project? Also, perhaps unrelated, what version of node are you using?

[barroudjo]

@kingcody node: 0.10.31, and I'm using the master branch of this project taken through an npm install on the 9th of september.
Edit: more precisely the version I use is: 2.0.13

[gintsgints]

How to repeat:

• login one window as admin
• open another window and logout and close
• click to admin menu at first one and login route/state changes. (but admin menu still visible)

[gintsgints]

Do not know is it best solution, but I propose changing login controller to logout for sure.
(because 401 response in client/app.js is reason of expiration, from where it is sent to login, while all local variables could show normal user state)

[gintsgints]

https://github.com/gintsgints/generator-angular-fullstack/commit/66b734b3400247a592a2f2164532c8e9a00d6768

For some reason I cannot (may be my hands guilty) to make PR if I have some more commits.

[lukeberry99]

Still noticing this problem, has it been merged?

[gintsgints]

Not yet merged. On the way.

[lukeberry99]

Great to know, cheers.