Projet

Général

Profil

Bug #5639

Seal leak of SamlKeyValue models

Ajouté par Benjamin Dauvergne il y a plus de 9 ans. Mis à jour il y a environ 9 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
Début:
07 octobre 2014
Echéance:
% réalisé:

100%

Temps estimé:
Patch proposed:
Oui
Planning:

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

Révision 75eac5f0 (diff)
Ajouté par Benjamin Dauvergne il y a plus de 9 ans

Remove LibertyFederation model (refs #5639)

The model is unused since commit df00c9d6c8.

Révision 9d760a1d (diff)
Ajouté par Benjamin Dauvergne il y a plus de 9 ans

Add an ExpireManager to share common code around expiration of models (refs #5639)

Révision d21ab765 (diff)
Ajouté par Benjamin Dauvergne il y a plus de 9 ans

Add created filed to SamlKeyValue to permit expiration of stored values (fixes #5639)

This commit also set the default manager to be the ExpireManager.

Révision 1d6a4d9e (diff)
Ajouté par Benjamin Dauvergne il y a environ 9 ans

Remove LibertyFederation model (refs #5639)

The model is unused since commit df00c9d6c8.

Révision d5675f25 (diff)
Ajouté par Benjamin Dauvergne il y a environ 9 ans

Add an ExpireManager to share common code around expiration of models (refs #5639)

Révision c38ee5e7 (diff)
Ajouté par Benjamin Dauvergne il y a environ 9 ans

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

#1

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).

#2

Mis à jour par Benjamin Dauvergne il y a plus de 9 ans

  • Statut changé de Nouveau à En cours
#3

Mis à jour par Benjamin Dauvergne il y a plus de 9 ans

  • Assigné à mis à Benjamin Dauvergne
#4

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.

#5

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
#6

Mis à jour par Benjamin Dauvergne il y a environ 9 ans

  • Version cible mis à 2.1.12
#7

Mis à jour par Benjamin Dauvergne il y a environ 9 ans

  • Statut changé de Résolu (à déployer) à Nouveau

It's not committed.

#8

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

#9

Mis à jour par Serghei Mihai il y a environ 9 ans

  • Patch proposed changé de Non à Oui
#11

Mis à jour par Benjamin Dauvergne il y a environ 9 ans

  • Fichier 0001-Remove-LibertyFederation-model-refs-5639.patch supprimé
#12

Mis à jour par Benjamin Dauvergne il y a environ 9 ans

  • Fichier 0002-Add-an-ExpireManager-to-share-common-code-around-exp.patch supprimé
#13

Mis à jour par Benjamin Dauvergne il y a environ 9 ans

  • Fichier 0003-Add-created-filed-to-SamlKeyValue-to-permit-expirati.patch supprimé
#14

Mis à jour par Serghei Mihai il y a environ 9 ans

Ack

#15

Mis à jour par Benjamin Dauvergne il y a environ 9 ans

  • Statut changé de Nouveau à Résolu (à déployer)
#16

Mis à jour par Benjamin Dauvergne il y a environ 9 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF