DependencyTrack / DependencyTrack/frontend
Stop using sessionStorage (security/usability issue)
- Dominant language
- Vue
- Stars
- 171
- Forks
- 250
- Avg merge
- 10h 56m
- Merged PRs (30d)
- 84
Description
The defect may already be reported! Please search for the defect before creating one.
### Current Behavior:
UI uses window.sessionStorage for auth token
### Expected Behavior:
It should be stored in HTTP Only Cookie (better with secure flag).
### Why?
- Session Storage / Local storage isn't considered to be a secure storage for auth tokens: https://auth0.com/blog/secure-browser-storage-the-facts/
- Now you can't effectively work with Dependency Track using several tabs as each tab has it's own Session Storage. Many users think that it's a bug as absolute majority of the websites share auth session between tabs.
Contributor guide
No contributing guide indexed for this repository
Research direction
No files or tests are named. Start by locating the UI's window.sessionStorage usage and tracing the auth-token flow; verify how the frontend receives and sends authentication state. Done means authentication uses an HTTP-only, preferably secure cookie and the session is shared across browser tabs.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- authentication, frontend, security
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100