Bug #5639
Seal leak of SamlKeyValue models
100%
Description
Authentic2 integrate a small key value datastore based on the SamlKeyValue model. In the future we should migrate to directly use the session contrib app for this storage as it provides the same features (persistent temporary storage of key value pairs). In the meantime we should allow key values to expire after 7 days, as it's only used to keep around Lasso objects when request handling needs to be interrupted for interaction with the user during login or consent requests. The expiration could be shorter but 7 days is just enough to not let the storage grow too much.
Fichiers
Révisions associées
Add an ExpireManager to share common code around expiration of models (refs #5639)
Add created filed to SamlKeyValue to permit expiration of stored values (fixes #5639)
This commit also set the default manager to be the ExpireManager.
Remove LibertyFederation model (refs #5639)
The model is unused since commit df00c9d6c8.
Add an ExpireManager to share common code around expiration of models (refs #5639)
Add created filed to SamlKeyValue to permit expiration of stored values (fixes #5639)
This commit also set the default manager to be the ExpireManager.
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 9 ans
- Fichier 0001-Remove-LibertyFederation-model-refs-5639.patch ajouté
- Fichier 0002-Add-an-ExpireManager-to-share-common-code-around-exp.patch ajouté
- Fichier 0003-Add-created-filed-to-SamlKeyValue-to-permit-expirati.patch ajouté
- Patch proposed changé de Non à Oui
The first patch also remove the LibertyAssertion model which is not used anymore and has been accumulating on some deployments of authentic (notable Montpellier, there is a 133 Mb saml_keyvalue table and a 30 Mb saml_libertyassertion table).
Mis à jour par Benjamin Dauvergne il y a environ 9 ans
- Statut changé de En cours à Nouveau
We should probably target the session storage now.
Mis à jour par Benjamin Dauvergne il y a environ 9 ans
- Statut changé de Nouveau à Résolu (à déployer)
- % réalisé changé de 0 à 100
Appliqué par commit d21ab765a5ab850ae2e2b0af67954f3a0478b735.
Mis à jour par Benjamin Dauvergne il y a environ 9 ans
- Statut changé de Résolu (à déployer) à Nouveau
It's not committed.
Mis à jour par Serghei Mihai il y a environ 9 ans
- Patch proposed changé de Oui à Non
The patches don't apply anymore as the source was moved under src
.
Is it possible to rebase them against master
Mis à jour par Benjamin Dauvergne il y a environ 9 ans
- Fichier 0001-Remove-LibertyFederation-model-refs-5639.patch 0001-Remove-LibertyFederation-model-refs-5639.patch ajouté
- Fichier 0002-Add-an-ExpireManager-to-share-common-code-around-exp.patch 0002-Add-an-ExpireManager-to-share-common-code-around-exp.patch ajouté
- Fichier 0003-Add-created-filed-to-SamlKeyValue-to-permit-expirati.patch 0003-Add-created-filed-to-SamlKeyValue-to-permit-expirati.patch ajouté
That's why I prefer pull requests, rebase is automatic when trivial. Anyway, new patches.
Mis à jour par Benjamin Dauvergne il y a environ 9 ans
- Fichier
0001-Remove-LibertyFederation-model-refs-5639.patchsupprimé
Mis à jour par Benjamin Dauvergne il y a environ 9 ans
- Fichier
0002-Add-an-ExpireManager-to-share-common-code-around-exp.patchsupprimé
Mis à jour par Benjamin Dauvergne il y a environ 9 ans
- Fichier
0003-Add-created-filed-to-SamlKeyValue-to-permit-expirati.patchsupprimé
Mis à jour par Benjamin Dauvergne il y a environ 9 ans
- Statut changé de Nouveau à Résolu (à déployer)
Appliqué par commit c38ee5e7c6139db2d91c18c9e162037f1c0689d0.
Mis à jour par Benjamin Dauvergne il y a environ 9 ans
- Statut changé de Résolu (à déployer) à Fermé
Remove LibertyFederation model (refs #5639)
The model is unused since commit df00c9d6c8.