Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Principal required #106

Closed
trydofor opened this issue Jun 30, 2023 · 2 comments
Closed

Principal required #106

trydofor opened this issue Jun 30, 2023 · 2 comments
Labels
bug :lady_beetle: Something isn't working

Comments

@trydofor
Copy link
Owner

trydofor commented Jun 30, 2023 •
edited 

java.lang.IllegalArgumentException: Principal required
 at org.springframework.util.Assert.notNull(Assert.java:201) ~[spring-core-5.3.23.jar:5.3.23]
 at org.springframework.security.core.session.SessionInformation.<init>(SessionInformation.java:53) ~[spring-security-core-5.6.3.jar:5.6.3]
 at pro.fessional.wings.slardar.session.WingsSessionInformation.<init>(WingsSessionInformation.java:26) ~[slardar-webmvc-2.6.6.210-20221209.075942-40.jar:2.6.6.210-SNAPSHOT]
 at pro.fessional.wings.slardar.session.WingsSessionRegistry.getSessionInformation(WingsSessionRegistry.java:77) ~[slardar-webmvc-2.6.6.210-20221209.075942-40.jar:2.6.6.210-SNAPSHOT]

anonymous user get session
@trydofor trydofor added the bug :lady_beetle: Something isn't working label Jun 30, 2023
@trydofor
Copy link
Owner Author

trydofor commented Jul 6, 2023

  • guest - AnonymousAuthenticationToken.principle=DefaultWingsUserDetails
  • user - UsernamePasswordAuthenticationToken.principle="anonymousUser"

@trydofor
Copy link
Owner Author

trydofor commented Jul 6, 2023

  • use SpringSessionBackedSessionRegistry, delete WingsSessionRegistry
  • OkHttpClientHelper add clearCookie
  • more testcases about session

trydofor added a commit that referenced this issue Jul 13, 2023
@trydofor
🐛 expose SpringSessionBackedSessionRegistry #106
2ac0f8a
@trydofor trydofor mentioned this issue Aug 24, 2023
Merged
trydofor added a commit that referenced this issue Aug 24, 2023
@trydofor
🐛 expose SpringSessionBackedSessionRegistry #106
2cd9fd3
trydofor added a commit that referenced this issue Aug 24, 2023
@trydofor
Revert "🐛 expose SpringSessionBackedSessionRegistry #106"
c69d99d 
This reverts commit 2cd9fd3.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug :lady_beetle: Something isn't working
Projects
WingsBoot Backlog
Archived in project
Milestone
No milestone
Development

No branches or pull requests
1 participant
@trydofor