spring-projects / spring-projects/spring-security
Consider preventing SecurityContextHolder.getContext().setAuthentication
Nobody has claimed this yet.
- Dominant language
- Java
- Stars
- 9.6k
- Forks
- 6.3k
- Avg merge
- 2d 11h
- Merged PRs (30d)
- 52
Description
Invoking this can cause race conditions and should not be done. We should consider preventing this behavior. See http://docs.spring.io/spring-security/site/docs/4.0.x/reference/htmlsingle/#tech-intro-sec-context-persistence
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.
Research direction
Start by reading the linked Spring Security 4.0 reference section on security-context persistence and identify how SecurityContextHolder.getContext().setAuthentication is currently used. The issue does not name files, tests, or a specific prevention mechanism; completion would require an agreed design for preventing the race-prone behavior and corresponding verification.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java, spring
- Domain
- authentication, security
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 20/100