https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342022-10-07T10:18:31ZRedmine Entr’ouvertPlugin FS FranceConnect - Development #69989: Section "Changement de mot de passe" - NOK (point 26)https://dev.entrouvert.org/issues/69989?journal_id=4127772022-10-07T10:18:31ZBrice Mallet
<ul><li><strong>Lié à</strong> <i><a class="issue tracker-2 status-5 priority-4 priority-default closed child" href="/issues/68360">Development #68360</a>: Cinématique de Réconciliation de compte - NOK</i> ajouté</li></ul> Plugin FS FranceConnect - Development #69989: Section "Changement de mot de passe" - NOK (point 26)https://dev.entrouvert.org/issues/69989?journal_id=4127792022-10-07T10:19:42ZBrice Mallet
<ul><li><strong>Lié à</strong> <i><a class="issue tracker-5 status-2 priority-4 priority-default parent" href="/issues/68358">Project management #68358</a>: Demandes d'évolution pour la nouvelle certification FranceConnect</i> ajouté</li></ul> Plugin FS FranceConnect - Development #69989: Section "Changement de mot de passe" - NOK (point 26)https://dev.entrouvert.org/issues/69989?journal_id=4242492022-11-22T14:00:11ZBenjamin Dauvergne
<ul><li><strong>Assigné à</strong> mis à <i>Benjamin Dauvergne</i></li></ul> Plugin FS FranceConnect - Development #69989: Section "Changement de mot de passe" - NOK (point 26)https://dev.entrouvert.org/issues/69989?journal_id=4242762022-11-22T14:35:59ZBenjamin Dauvergne
<ul><li><strong>Fichier</strong> <a href="/attachments/72133">0001-auth_fc-store-id_token-in-session-when-linking-69989.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/72133/0001-auth_fc-store-id_token-in-session-when-linking-69989.patch">0001-auth_fc-store-id_token-in-session-when-linking-69989.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/72134">0002-auth_fc-show-warning-on-password-change-page-if-user.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/72134/0002-auth_fc-show-warning-on-password-change-page-if-user.patch">0002-auth_fc-show-warning-on-password-change-page-if-user.patch</a> ajouté</li><li><strong>Tracker</strong> changé de <i>Project management</i> à <i>Development</i></li><li><strong>Statut</strong> changé de <i>Nouveau</i> à <i>Solution proposée</i></li><li><strong>Patch proposed</strong> changé de <i>Non</i> à <i>Oui</i></li></ul>Ceinture et bretelle :
<ul>
<li>la vue de mot de passe ne sera plus visible dans le cas particulier où on s'est connecté avec FranceConnect, puis délier, puis relier.</li>
<li>si jamais le compte courant est relié à FranceConnect mais que la vue de changement de mot de passe est visible (ex.: on s'est connecté via son email/mot de passe et pas par FranceConnect) alors on affiche un disclaimer.</li>
</ul> Plugin FS FranceConnect - Development #69989: Section "Changement de mot de passe" - NOK (point 26)https://dev.entrouvert.org/issues/69989?journal_id=4242792022-11-22T14:37:48ZBenjamin Dauvergne
<ul><li><strong>Fichier</strong> <a href="/attachments/72135">0001-auth_fc-store-id_token-in-session-when-linking-69989.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/72135/0001-auth_fc-store-id_token-in-session-when-linking-69989.patch">0001-auth_fc-store-id_token-in-session-when-linking-69989.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/72136">0003-translation-update-69989.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/72136/0003-translation-update-69989.patch">0003-translation-update-69989.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/72137">0002-auth_fc-show-warning-on-password-change-page-if-user.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/72137/0002-auth_fc-show-warning-on-password-change-page-if-user.patch">0002-auth_fc-show-warning-on-password-change-page-if-user.patch</a> ajouté</li></ul><p>Avec la trad.</p> Plugin FS FranceConnect - Development #69989: Section "Changement de mot de passe" - NOK (point 26)https://dev.entrouvert.org/issues/69989?journal_id=4242832022-11-22T14:41:40ZSerghei Mihai
<ul></ul><pre>
def get(self, request, *args, **kwargs):
return super().get(request, *args, **kwargs)
</pre>
<p>n'est pas inutile ?</p> Plugin FS FranceConnect - Development #69989: Section "Changement de mot de passe" - NOK (point 26)https://dev.entrouvert.org/issues/69989?journal_id=4243502022-11-22T20:20:43ZBenjamin Dauvergne
<ul><li><strong>Fichier</strong> <a href="/attachments/72143">0001-auth_fc-store-id_token-in-session-when-linking-69989.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/72143/0001-auth_fc-store-id_token-in-session-when-linking-69989.patch">0001-auth_fc-store-id_token-in-session-when-linking-69989.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/72144">0003-translation-update-69989.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/72144/0003-translation-update-69989.patch">0003-translation-update-69989.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/72145">0002-auth_fc-show-warning-on-password-change-page-if-user.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/72145/0002-auth_fc-show-warning-on-password-change-page-if-user.patch">0002-auth_fc-show-warning-on-password-change-page-if-user.patch</a> ajouté</li></ul> Plugin FS FranceConnect - Development #69989: Section "Changement de mot de passe" - NOK (point 26)https://dev.entrouvert.org/issues/69989?journal_id=4244462022-11-23T09:41:36ZPaul Marillonnet
<ul></ul><p>Dans 0001 peut-être moyen de factoriser du code avec ce qui est fait dans le cas où il y a login et non pas liaison dans le code à la fin <code>handle_authorization_response</code><br /><pre><code class="python syntaxhl"> <span class="k">if</span> <span class="n">request</span><span class="p">.</span><span class="n">user</span><span class="p">.</span><span class="n">is_authenticated</span><span class="p">:</span>
<span class="k">return</span> <span class="bp">self</span><span class="p">.</span><span class="n">link</span><span class="p">(</span><span class="n">request</span><span class="p">)</span>
<span class="k">else</span><span class="p">:</span>
<span class="k">return</span> <span class="bp">self</span><span class="p">.</span><span class="n">login</span><span class="p">(</span><span class="n">request</span><span class="p">)</span>
</code></pre> ?<br />Par exemple dans le cas login je vois qu’en plus il y a paramétrage de l’expiration de la session, après stockage de l’ID token :<br /><pre><code class="python syntaxhl"> <span class="c1"># set session expiration policy to EXPIRE_AT_BROWSER_CLOSE
</span> <span class="n">request</span><span class="p">.</span><span class="n">session</span><span class="p">.</span><span class="n">set_expiry</span><span class="p">(</span><span class="mi">0</span><span class="p">)</span>
</code></pre> (dans finish_login) qui n’est pas repris dans 0001 ?</p>
<p>0002 ok, peut-être juste<br />· s/not the one from the account used through FranceConnect/not the one from your FranceConnect provider/<br />· s/connect with locally/log in locally/<br />dans le message affiché à l’usager ?</p> Plugin FS FranceConnect - Development #69989: Section "Changement de mot de passe" - NOK (point 26)https://dev.entrouvert.org/issues/69989?journal_id=4244542022-11-23T09:46:40ZBenjamin Dauvergne
<ul></ul><p>Paul Marillonnet a écrit :</p>
<blockquote>
<p>Dans 0001 peut-être moyen de factoriser du code avec ce qui est fait dans le cas où il y a login et non pas liaison dans le code à la fin <code>handle_authorization_response</code><br />[...] ?<br />Par exemple dans le cas login je vois qu’en plus il y a paramétrage de l’expiration de la session, après stockage de l’ID token :<br />[...] (dans finish_login) qui n’est pas repris dans 0001 ?</p>
</blockquote>
<p>Ça ne concerne qu'une session ouverte par FranceConnect, la session ayant pu être ouverte via mot de passe, je préfère ne pas changer (<a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: auth_fc: passer la session en EXPIRE_AT_BROWSER_CLOSE (cookie de session sans expiration) (Fermé)" href="https://dev.entrouvert.org/issues/40479">#40479</a>).</p>
<blockquote>
<p>0002 ok, peut-être juste<br />· s/not the one from the account used through FranceConnect/not the one from your FranceConnect provider/<br />· s/connect with locally/log in locally/<br />dans le message affiché à l’usager ?</p>
</blockquote>
<p>Ok.</p> Plugin FS FranceConnect - Development #69989: Section "Changement de mot de passe" - NOK (point 26)https://dev.entrouvert.org/issues/69989?journal_id=4244562022-11-23T09:48:44ZBenjamin Dauvergne
<ul><li><strong>Fichier</strong> <a href="/attachments/72148">0001-auth_fc-store-id_token-in-session-when-linking-69989.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/72148/0001-auth_fc-store-id_token-in-session-when-linking-69989.patch">0001-auth_fc-store-id_token-in-session-when-linking-69989.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/72149">0003-translation-update-69989.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/72149/0003-translation-update-69989.patch">0003-translation-update-69989.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/72150">0002-auth_fc-show-warning-on-password-change-page-if-user.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/72150/0002-auth_fc-show-warning-on-password-change-page-if-user.patch">0002-auth_fc-show-warning-on-password-change-page-if-user.patch</a> ajouté</li></ul><p>Voilà.</p> Plugin FS FranceConnect - Development #69989: Section "Changement de mot de passe" - NOK (point 26)https://dev.entrouvert.org/issues/69989?journal_id=4244592022-11-23T09:52:27ZPaul Marillonnet
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Solution validée</i></li></ul><p>Benjamin Dauvergne a écrit :</p>
<blockquote>
<p>Ça ne concerne qu'une session ouverte par FranceConnect, la session ayant pu être ouverte via mot de passe, je préfère ne pas changer (<a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: auth_fc: passer la session en EXPIRE_AT_BROWSER_CLOSE (cookie de session sans expiration) (Fermé)" href="https://dev.entrouvert.org/issues/40479">#40479</a>).</p>
</blockquote>
<p>Ok.</p>
<blockquote>
<p>Ok.</p>
</blockquote>
<p>Ok, peut-être juste un s/mail/courriel/ dans le message traduit dans 0003 puisque cela semble être la convention dans authentic.</p>
<p>Ack.</p> Plugin FS FranceConnect - Development #69989: Section "Changement de mot de passe" - NOK (point 26)https://dev.entrouvert.org/issues/69989?journal_id=4245122022-11-23T10:56:29ZBenjamin Dauvergne
<ul><li><strong>Fichier</strong> <a href="/attachments/72152">0001-auth_fc-store-id_token-in-session-when-linking-69989.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/72152/0001-auth_fc-store-id_token-in-session-when-linking-69989.patch">0001-auth_fc-store-id_token-in-session-when-linking-69989.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/72153">0003-translation-update-69989.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/72153/0003-translation-update-69989.patch">0003-translation-update-69989.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/72154">0002-auth_fc-show-warning-on-password-change-page-if-user.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/72154/0002-auth_fc-show-warning-on-password-change-page-if-user.patch">0002-auth_fc-show-warning-on-password-change-page-if-user.patch</a> ajouté</li><li><strong>Statut</strong> changé de <i>Solution validée</i> à <i>Solution proposée</i></li></ul><p>Oublié de pousser le test.</p> Plugin FS FranceConnect - Development #69989: Section "Changement de mot de passe" - NOK (point 26)https://dev.entrouvert.org/issues/69989?journal_id=4245202022-11-23T11:39:23ZPaul Marillonnet
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Solution validée</i></li></ul><p>Benjamin Dauvergne a écrit :</p>
<blockquote>
<p>Oublié de pousser le test.</p>
</blockquote>
<p>Magnifique. Reste juste pylint à contenter :<br /><pre>
tests/auth_fc/test_views.py:25: [C1802(use-implicit-booleaness-not-len), test_password_change_view_with_fc] Do not use `len(SEQUENCE)` without comparison to determine if a sequence is empty
</pre><br />et c’est bon.</p> Plugin FS FranceConnect - Development #69989: Section "Changement de mot de passe" - NOK (point 26)https://dev.entrouvert.org/issues/69989?journal_id=4255812022-11-28T14:05:32ZBenjamin Dauvergne
<ul><li><strong>Statut</strong> changé de <i>Solution validée</i> à <i>Résolu (à déployer)</i></li></ul><pre>commit a8ebccc5aa8ec83f9db0e0edee8b309be40def28
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Tue Nov 22 15:37:37 2022 +0100
translation update (#69989)
commit 9171c30febcb58c6af4a2514ebfbcda64d8b33db
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Tue Nov 22 15:34:26 2022 +0100
auth_fc: show warning on password change page if user is linked to FranceConnect (#69989)
commit 89b526066d6373e741c4843c6c93632febc8d2cb
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Tue Nov 22 15:08:07 2022 +0100
auth_fc: store id_token in session when linking (#69989)
Password change view is hidden only if the current session was opened
with FranceConnect and if the id_token is visible in the session, this
change will enforce the second condition when during the same session
the account is unlinked and linked again to FranceConnect.
It will also permit to launch a logout if the link fails.</pre> Plugin FS FranceConnect - Development #69989: Section "Changement de mot de passe" - NOK (point 26)https://dev.entrouvert.org/issues/69989?journal_id=4322462022-12-23T09:15:33ZTransition automatique
<ul><li><strong>Statut</strong> changé de <i>Résolu (à déployer)</i> à <i>Solution déployée</i></li></ul> Plugin FS FranceConnect - Development #69989: Section "Changement de mot de passe" - NOK (point 26)https://dev.entrouvert.org/issues/69989?journal_id=4467392023-02-26T03:42:01ZTransition automatique
<ul></ul><p>Automatic expiration</p>