Get automatically re-logged-in without
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 35/100
- Issue type
- Bug
- Clarity
- Needs clarification
- Activity status
- Stale
- Tech stack
- javascript
- Domain
- authentication
Research direction
The issue names only the authentication code and logout button, not specific files or tests. Start by tracing the logout flow and checking how cookies and local storage are handled; done should mean logging out clears the relevant state so the next login can select a different user.
Written by the indexing model from the issue text.
Description
I have a problem with loggin in to NSS that it assumes which user I want to be instead of waiting for me to type it in. For example whenever I log in with inrupt.net it logs me in as timblBOT.inrupt.net not timbl.
@jaxoncreed Not sure where to report that. I guess it is a problem with the current auth code which noone is working on, presumably — though who knows it may carry through to the next version too.
Tim Berners-Lee: Yes @RubenVerborgh it looks like a cookie problem that it sees the cookie and just grabs it. But if I explicitly log out it should delete the cookie surely
Ruben Verborgh: Indeed, that is what NSS should do.
Tim Berners-Lee @timbl 13:06
Suppose I try delete cookies > log out > … weird the log out button doesn’t do anything.
well not surprising it should break solidos as I didn’t tell it I logged out by myself.
But I can reload the page which then takes me to a logged out state, then I can go log in > select inrupt -> then I have a choice of user. So the problem seems to be that NSS doesn’t delete all Local Storage and Cookies on logout?
I’ll file it as an NSS bug then.
I suggest it is good to delete ALL local storage and ALL cookies for the domain partly form privacy and partly to help move on to the next version of the auth protocol which may format these things differently.
- Dominant language
- JavaScript
- Stars
- 1.8k
- Forks
- 308
- PR merge metrics
- No merged PRs in 30d
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from nodeSolidServer/node-solid-server
-
Difficulty 1/5 Under an hour Newbie friendliness 72/100
nodeSolidServer/node-solid-server#1848 · 2 comments · 1 reaction ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
nodeSolidServer/node-solid-server#1841 · 2 comments ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
nodeSolidServer/node-solid-server#1147 ·
-
Difficulty 4/5 3-5 days Newbie friendliness 35/100
nodeSolidServer/node-solid-server#1867 ·
-
Difficulty 5/5 Over a week Newbie friendliness 45/100
nodeSolidServer/node-solid-server#1853 ·
All issues in nodeSolidServer/node-solid-server
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
-
area-deployment area-integrations triage:bot-seen
Difficulty 2/5 Half a day Newbie friendliness 86/100
-
Issue-Bug
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
sugarlabs/musicblocks#8924 ·
-
Difficulty 1/5 Under an hour Newbie friendliness 90/100
ArduPilot/ardupilot_wiki#8088 ·
-
[BUG] createTool tools cannot be registered with Mastra when exactOptionalPropertyTypes is enabled Opencustomer-eng status: needs triage
Difficulty 2/5 1-3 hours Newbie friendliness 84/100