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.

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

Improve clean up stored procedures to remove expired data in IDN_AUTH_TEMP_SESSION_STORE #22080

Open
Dinithi-Hasanika opened this issue Dec 19, 2024 · 0 comments

Comments

@Dinithi-Hasanika
Copy link

Dinithi-Hasanika commented Dec 19, 2024

Current Limitation

In identity server 7.0.0, the session and temp data separation is enabled by default [1]. Because of this temporary authentication context data is inserted to IDN_AUTH_TEMP_SESSION_STORE. When using stored procedures for session data clean up [2], expired data in IDN_AUTH_TEMP_SESSION_STORE table is not removed. This leads to IDN_AUTH_TEMP_SESSION_STORE table to take unnecessary huge space.

[1] - wso2/carbon-identity-framework#5335
[2] - https://is.docs.wso2.com/en/latest/deploy/configure/databases/data-purging/

Suggested Improvement

it is required to improve the clean up stored procedures to remove expired data in IDN_AUTH_TEMP_SESSION_STORE table as well. This has been addressed for mysql[1]. Similarly this needs to be improved for other databases as well, for an example mssql [2].

[1] - wso2/carbon-identity-framework#4568
[2] - https://github.com/wso2/carbon-identity-framework/blob/master/features/identity-core/org.wso2.carbon.identity.core.server.feature/resources/dbscripts/stored-procedures/mssql/sessiondata-cleanup/mssql-session-data-cleanup.sql

Version

Identity server 7.0.0

@Dinithi-Hasanika Dinithi-Hasanika changed the title Improve mssql clean up stored procedure to remove expired data in IDN_AUTH_TEMP_SESSION_STORE Improve clean up stored procedures to remove expired data in IDN_AUTH_TEMP_SESSION_STORE Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant