dasch-swiss / dsp-api

DaSCH Service Platform API
http://dasch-swiss.github.io/dsp-api/
Apache License 2.0
74 stars 18 forks source link

no logout #804

Open loicjaouen opened 6 years ago

loicjaouen commented 6 years ago

When salsah 1.5 logs out:

The browser should have deleted it... it doesn't.

That's on firefox, chrome and safari, I didn't test others.

subotic commented 6 years ago

AFAIK the session gets deleted on the webapi server, so when the client sends the "old" session id, he will not be authenticated. Why the cookie is not deleted on the client I don't know. Maybe the specs have changed. Anyway, this is old school stuff and shouldn't be used today anymore.

Also, sessions are only used in v1 authentication. In v2 authentication, tokens are used, which Salsah2 is using.

loicjaouen commented 6 years ago

the session gets deleted on the webapi server, so when the client sends the "old" session id, he will not be authenticated

Exactly, that's how the problem showed up, users logging out and getting unauthorized access which they did not understand as they were just logging out to demonstrate the platform to external people.

Anyway, this is old school stuff and shouldn't be used today anymore.

the whole salsah 1.5 is old school and while it shouldn't be used anymore, it's the only production front-end we currently have to offer.

loicjaouen commented 6 years ago

Why the cookie is not deleted on the client I don't know.

neither do I 😢

loicjaouen commented 6 years ago

The procedure seems completely legitimate.

What should be tested is: