https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342023-03-15T16:50:23ZRedmine Entr’ouvertAuthentic 2 - Bug #75474: auth_oidc : le choix des attributs de mapping est extensif mais l’initialisation de ceux-ci ne retient que les attributs actifs https://dev.entrouvert.org/issues/75474?journal_id=4512642023-03-15T16:50:23ZPaul Marillonnet
<ul><li><strong>Statut</strong> changé de <i>Nouveau</i> à <i>En cours</i></li><li><strong>Assigné à</strong> mis à <i>Paul Marillonnet</i></li></ul><p>Je regarde.</p> Authentic 2 - Bug #75474: auth_oidc : le choix des attributs de mapping est extensif mais l’initialisation de ceux-ci ne retient que les attributs actifs https://dev.entrouvert.org/issues/75474?journal_id=4514512023-03-16T10:28:21ZRobot Gitea
<ul><li><strong>Statut</strong> changé de <i>En cours</i> à <i>Solution proposée</i></li></ul><p>Paul Marillonnet (pmarillonnet) a ouvert une pull request sur Gitea concernant cette demande :</p>
<ul>
<li>URL : <a class="external" href="https://git.entrouvert.org/entrouvert/authentic/pulls/24">https://git.entrouvert.org/entrouvert/authentic/pulls/24</a></li>
<li>Titre : auth_oidc: exclude disabled attributes in claim mapping form (<a class="issue tracker-1 status-5 priority-4 priority-default closed" title="Bug: auth_oidc : le choix des attributs de mapping est extensif mais l’initialisation de ceux-ci ne re... (Fermé)" href="https://dev.entrouvert.org/issues/75474">#75474</a>)</li>
<li>Modifications : <a class="external" href="https://git.entrouvert.org/entrouvert/authentic/pulls/24/files">https://git.entrouvert.org/entrouvert/authentic/pulls/24/files</a></li>
</ul> Authentic 2 - Bug #75474: auth_oidc : le choix des attributs de mapping est extensif mais l’initialisation de ceux-ci ne retient que les attributs actifs https://dev.entrouvert.org/issues/75474?journal_id=4515492023-03-16T11:55:49ZBenjamin Dauvergne
<ul></ul><p>Ok mais ça ne va pas corriger le bug, il doit y avoir une partie dans authentic2_auth_oidc.backends pour ignorer les règles existantes et aussi peut-être dans OIDCProvider.perform_synchronisation() (bêtement en filtrant OIDCClaimMapping sur attribute en excluant les attributs désactivés).</p> Authentic 2 - Bug #75474: auth_oidc : le choix des attributs de mapping est extensif mais l’initialisation de ceux-ci ne retient que les attributs actifs https://dev.entrouvert.org/issues/75474?journal_id=4515502023-03-16T11:57:48ZBenjamin Dauvergne
<ul></ul><p>Et patch actuel pas nécessaire, ils sont déjà filtré sur <code>.objects</code> :<br /><pre>
objects = managers.AttributeManager(disabled=False)
</pre><br />le problème est vraiment uniquement dans backends.py.</p> Authentic 2 - Bug #75474: auth_oidc : le choix des attributs de mapping est extensif mais l’initialisation de ceux-ci ne retient que les attributs actifs https://dev.entrouvert.org/issues/75474?journal_id=4516002023-03-16T13:56:07ZPaul Marillonnet
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>En cours</i></li></ul><p>Benjamin Dauvergne a écrit :</p>
<blockquote>
<p>Et patch actuel pas nécessaire, ils sont déjà filtré sur <code>.objects</code> :<br />[...]<br />le problème est vraiment uniquement dans backends.py.</p>
</blockquote>
<p>Ah oui alors je n’ai pas compris comment la trace était survenue. Je vais reprendre de zéro pour tenter de capter le truc.</p> Authentic 2 - Bug #75474: auth_oidc : le choix des attributs de mapping est extensif mais l’initialisation de ceux-ci ne retient que les attributs actifs https://dev.entrouvert.org/issues/75474?journal_id=4516262023-03-16T14:22:51ZBenjamin Dauvergne
<ul></ul><p>Ben si tu regardes la trace elle est bien dans backends et les mappings datent forcément d'avant la désactivation du champ.</p> Authentic 2 - Bug #75474: auth_oidc : le choix des attributs de mapping est extensif mais l’initialisation de ceux-ci ne retient que les attributs actifs https://dev.entrouvert.org/issues/75474?journal_id=4516282023-03-16T14:24:01ZPaul Marillonnet
<ul></ul><p>Benjamin Dauvergne a écrit :</p>
<blockquote>
<p>les mappings datent forcément d'avant la désactivation du champ.</p>
</blockquote>
<p>Oui j’ai mis du temps à comprendre cela, merci :)</p> Authentic 2 - Bug #75474: auth_oidc : le choix des attributs de mapping est extensif mais l’initialisation de ceux-ci ne retient que les attributs actifs https://dev.entrouvert.org/issues/75474?journal_id=4516542023-03-16T14:49:19ZPaul Marillonnet
<ul><li><strong>Statut</strong> changé de <i>En cours</i> à <i>Solution proposée</i></li></ul><p>Voilà, je veux bien juste laisser le test de l’ancien commit (parce que sans cela je sens que dans trois mois j’aurais à nouveau oublié cette histoire d’object manager qui filtre en douce les attributs activés).</p> Authentic 2 - Bug #75474: auth_oidc : le choix des attributs de mapping est extensif mais l’initialisation de ceux-ci ne retient que les attributs actifs https://dev.entrouvert.org/issues/75474?journal_id=4587402023-04-15T09:24:12ZRobot Gitea
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>En cours</i></li></ul><p>Benjamin Dauvergne (bdauvergne) a relu et demandé des modifications sur une pull request sur Gitea concernant cette demande :</p>
<ul>
<li>URL : <a class="external" href="https://git.entrouvert.org/entrouvert/authentic/pulls/24">https://git.entrouvert.org/entrouvert/authentic/pulls/24</a></li>
</ul> Authentic 2 - Bug #75474: auth_oidc : le choix des attributs de mapping est extensif mais l’initialisation de ceux-ci ne retient que les attributs actifs https://dev.entrouvert.org/issues/75474?journal_id=4800162023-07-17T14:40:27ZRobot Gitea
<ul><li><strong>Statut</strong> changé de <i>En cours</i> à <i>Solution proposée</i></li></ul> Authentic 2 - Bug #75474: auth_oidc : le choix des attributs de mapping est extensif mais l’initialisation de ceux-ci ne retient que les attributs actifs https://dev.entrouvert.org/issues/75474?journal_id=4800172023-07-17T14:40:28ZRobot Gitea
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Solution validée</i></li></ul><p>Serghei Mihai (smihai) a approuvé une pull request sur Gitea concernant cette demande :</p>
<ul>
<li>URL : <a class="external" href="https://git.entrouvert.org/entrouvert/authentic/pulls/24">https://git.entrouvert.org/entrouvert/authentic/pulls/24</a></li>
</ul> Authentic 2 - Bug #75474: auth_oidc : le choix des attributs de mapping est extensif mais l’initialisation de ceux-ci ne retient que les attributs actifs https://dev.entrouvert.org/issues/75474?journal_id=4800192023-07-17T14:49:48ZRobot Gitea
<ul><li><strong>Statut</strong> changé de <i>Solution validée</i> à <i>Résolu (à déployer)</i></li></ul><p>Paul Marillonnet (pmarillonnet) a mergé une pull request sur Gitea concernant cette demande :</p>
<ul>
<li>URL : <a class="external" href="https://git.entrouvert.org/entrouvert/authentic/pulls/24">https://git.entrouvert.org/entrouvert/authentic/pulls/24</a></li>
<li>Titre : auth_oidc: exclude disabled attributes in claim mapping form (<a class="issue tracker-1 status-5 priority-4 priority-default closed" title="Bug: auth_oidc : le choix des attributs de mapping est extensif mais l’initialisation de ceux-ci ne re... (Fermé)" href="https://dev.entrouvert.org/issues/75474">#75474</a>)</li>
<li>Modifications : <a class="external" href="https://git.entrouvert.org/entrouvert/authentic/pulls/24/files">https://git.entrouvert.org/entrouvert/authentic/pulls/24/files</a></li>
</ul> Authentic 2 - Bug #75474: auth_oidc : le choix des attributs de mapping est extensif mais l’initialisation de ceux-ci ne retient que les attributs actifs https://dev.entrouvert.org/issues/75474?journal_id=4805722023-07-19T15:14:30ZTransition automatique
<ul><li><strong>Statut</strong> changé de <i>Résolu (à déployer)</i> à <i>Solution déployée</i></li></ul> Authentic 2 - Bug #75474: auth_oidc : le choix des attributs de mapping est extensif mais l’initialisation de ceux-ci ne retient que les attributs actifs https://dev.entrouvert.org/issues/75474?journal_id=4924172023-09-24T02:42:01ZTransition automatique
<ul></ul><p>Automatic expiration</p>