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

SessionRepositoryFilterConfiguration can cause early initialization of SessionRepository beans including Redis #35240

Closed
philwebb opened this issue May 2, 2023 · 1 comment
Closed

SessionRepositoryFilterConfiguration can cause early initialization of SessionRepository beans including Redis #35240

philwebb opened this issue May 2, 2023 · 1 comment
Assignees
@philwebb
Labels
type: bug A general bug
Milestone
2.7.12

Comments

@philwebb
Copy link
Member

philwebb commented May 2, 2023

SessionRepositoryFilterConfiguration.sessionRepositoryFilterRegistration is a filter registration bean which means it's initialized very early. It injects a SessionRepositoryFilter which is defined in SpringHttpSessionConfiguration and itself injects a SessionRepository. If a RedisSessionRepository is being used, then full initialization of Redis occurs.
@philwebb philwebb self-assigned this May 2, 2023
@philwebb philwebb added the type: bug A general bug label May 2, 2023
@philwebb philwebb added this to the 2.7.x milestone May 2, 2023
@philwebb
Copy link
Member Author

philwebb commented May 2, 2023

This is similar to #4154

@philwebb philwebb changed the title SessionRepositoryFilterConfiguration can cause early initialziation of SessionRepository beans including Redis SessionRepositoryFilterConfiguration can cause early initialization of SessionRepository beans including Redis May 2, 2023
@philwebb philwebb mentioned this issue May 2, 2023
Closed
@philwebb philwebb modified the milestones: 2.7.x, 2.7.12 May 2, 2023
@Netyyyy Netyyyy mentioned this issue May 25, 2023
Closed
@Netyyyy Netyyyy mentioned this issue Jun 19, 2023
Closed
yhcast0 pushed a commit to Kyligence/kylin that referenced this issue Aug 13, 2023
@Mrhs121 @yhcast0
AL-8554 Fix bug that all node can not get sessionid from http requert…
a91d38a 
… which redirect by query node caused by commit 'prevent early initialization of SessionRepository beans' from spring community (#30507)

Fix bug that all node can not get sessionid from http requert which redirect by query node caused by commit 'prevent early initialization of SessionRepository beans' from spring community
spring-projects/spring-boot#35240
Aligadodo pushed a commit to Aligadodo/kylin that referenced this issue Nov 15, 2023
@Mrhs121
AL-8554 Fix bug that all node can not get sessionid from http requert…
abf4964 
… which redirect by query node caused by commit 'prevent early initialization of SessionRepository beans' from spring community (#30507)

Fix bug that all node can not get sessionid from http requert which redirect by query node caused by commit 'prevent early initialization of SessionRepository beans' from spring community
spring-projects/spring-boot#35240
Aligadodo pushed a commit to Aligadodo/kylin that referenced this issue Nov 17, 2023
@Mrhs121
AL-8554 Fix bug that all node can not get sessionid from http requert…
51d4269 
… which redirect by query node caused by commit 'prevent early initialization of SessionRepository beans' from spring community (#30507)

Fix bug that all node can not get sessionid from http requert which redirect by query node caused by commit 'prevent early initialization of SessionRepository beans' from spring community
spring-projects/spring-boot#35240
Aligadodo pushed a commit to Aligadodo/kylin that referenced this issue Nov 17, 2023
@Mrhs121
KYLIN-5720 Fix bug that all node can not get sessionid from http requ…
3592245 
…ert which redirect by query node caused by commit 'prevent early initialization of SessionRepository beans' from spring community

Fix bug that all node can not get sessionid from http requert which redirect by query node caused by commit 'prevent early initialization of SessionRepository beans' from spring community
spring-projects/spring-boot#35240
hit-lacus pushed a commit to apache/kylin that referenced this issue Nov 21, 2023
@Mrhs121 @hit-lacus
KYLIN-5720 Fix bug that all node can not get sessionid from http requ…
db19826 
…ert which redirect by query node caused by commit 'prevent early initialization of SessionRepository beans' from spring community

Fix bug that all node can not get sessionid from http requert which redirect by query node caused by commit 'prevent early initialization of SessionRepository beans' from spring community
spring-projects/spring-boot#35240
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@philwebb philwebb

Labels
type: bug A general bug
Projects
None yet
Milestone
2.7.12
Development

No branches or pull requests
1 participant
@philwebb