Projet

Général

Profil

Bug #62702

api de synchronisation : bug sur la réversibilité d’un sub en uuid pour un compte personne morale

Ajouté par Paul Marillonnet il y a environ 2 ans. Mis à jour il y a environ 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Paul Marillonnet
Catégorie:
-
Version cible:
-
Début:
11 mars 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

L’idée dans #58556 était que l’on suffixe l’uuid avec l’identifiant de profil personne morale choisi par l’usager, et qu’à chaque tentative de revert du sub en uuid on tente de retirer le suffixe.
Cette partie est je crois boguée (à confirmer, je vais tenter de reproduire), et affecte les subs issus d’uuid d’usagers ne possédant pas de profil PM.

Fichiers

0001-idp_oidc-fix-profile-suffix-during-sub-generation-62.patch (5,56 ko) 0001-idp_oidc-fix-profile-suffix-during-sub-generation-62.patch Paul Marillonnet, 11 mars 2022 16:37
0001-idp_oidc-fix-profile-suffix-during-sub-generation-62.patch (5,66 ko) 0001-idp_oidc-fix-profile-suffix-during-sub-generation-62.patch Paul Marillonnet, 16 mars 2022 12:13

Révisions associées

Révision 69f8e3f4 (diff)
Ajouté par Paul Marillonnet il y a environ 2 ans

idp_oidc: fix profile suffix during sub generation (#62702)

Historique

#2

Mis à jour par Paul Marillonnet il y a environ 2 ans

#3

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

Je ne comprends pas comment le split(b'#') fait après l'appel n'a pas marché, tu as une explication ?

#7

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

Plutôt se baser sur la longueur, on est certain qu'un uuid fait 16 octets, donc si c'est plus long c'est qu'il y autre chose après.

#8

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

  • Statut changé de Solution proposée à En cours
  • Assigné à mis à Paul Marillonnet
#9

Mis à jour par Paul Marillonnet il y a environ 2 ans

  • Sujet changé de gestion des personnes morales : bug sur la réversibilité d’un sub en uuid pour un compte personne morale (?) à api de synchronisation : bug sur la réversibilité d’un sub en uuid pour un compte personne morale
#10

Mis à jour par Paul Marillonnet il y a environ 2 ans

Benjamin Dauvergne a écrit :

Plutôt se baser sur la longueur, on est certain qu'un uuid fait 16 octets, donc si c'est plus long c'est qu'il y autre chose après.

D’accord, ce test sur la longueur, côté inversion du sub, en revanche côté génération du sub, je serais pour conserver le même suffixe. Nouvelle proposition de patch, corrigé en conséquence.

#11

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

  • Statut changé de Solution proposée à Solution validée

Ok.

#12

Mis à jour par Paul Marillonnet il y a environ 2 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 69f8e3f46cca39206042437f64b7306fc627e2d9
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date:   Fri Mar 11 16:36:36 2022 +0100

    idp_oidc: fix profile suffix during sub generation (#62702)
#13

Mis à jour par Transition automatique il y a environ 2 ans

  • Statut changé de Résolu (à déployer) à Solution déployée
#14

Mis à jour par Transition automatique il y a presque 2 ans

Automatic expiration

Formats disponibles : Atom PDF