Skip to content

Almost impossible to extend the RedisIndexedSessionRepository class #1562

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
Closed
j75 opened this issue Jan 24, 2020 · 1 comment
Closed

Almost impossible to extend the RedisIndexedSessionRepository class #1562

j75 opened this issue Jan 24, 2020 · 1 comment
Assignees
@marcusdacoregio
Labels
status: duplicate A duplicate of another issue

Comments

@j75
Copy link

j75 commented Jan 24, 2020 •
edited
Loading

Although not marked as final, the class RedisIndexedSessionRepository could not be easily extended ; more precisely, the createSession() could not be overriden, as the RedisSession class is not visible.
What about modifying the signature of this class to return a Session instance?
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jan 24, 2020
@marcusdacoregio marcusdacoregio added status: duplicate A duplicate of another issue and removed status: waiting-for-triage An issue we've not yet triaged labels Aug 8, 2023
@marcusdacoregio
Copy link
Contributor

Duplicate of #2258

@marcusdacoregio marcusdacoregio marked this as a duplicate of #2258 Aug 8, 2023
@marcusdacoregio marcusdacoregio self-assigned this Aug 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@marcusdacoregio marcusdacoregio

Labels
status: duplicate A duplicate of another issue
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@j75 @marcusdacoregio @spring-projects-issues