entrouvert
/
authentic
16
0
Fork 0
Code Issues Pull Requests 32 Releases Activity

Nettoyage de quelques settings obsolètes (#78230) #70

Merged
vdeniaud merged 1 commits from wip/78230-Nettoyage-de-quelques-settings-o into main 2023-06-12 15:55:42 +02:00
Conversation 1 Commits 1 Files Changed 2 +2 -10
vdeniaud commented 2023-06-07 11:18:27 +02:00
Owner
Copy Link

(j'ai tenté la suppression de A2_REGISTRATION_FORM_USERNAME_REGEX mais je laisse pour une autre fois, un peu trop particulier puisque dans nos déploiements on n'affiche même pas de champ username)

(j'ai tenté la suppression de A2_REGISTRATION_FORM_USERNAME_REGEX mais je laisse pour une autre fois, un peu trop particulier puisque dans nos déploiements on n'affiche même pas de champ username)
vdeniaud force-pushed wip/78230-Nettoyage-de-quelques-settings-o from 1a7ba07577 to 8016152c1d 2023-06-07 11:33:05 +02:00 Compare
vdeniaud changed title from Nettoyage de quelques settings obsolètes (#78230) to WIP: Nettoyage de quelques settings obsolètes (#78230) 2023-06-07 11:44:30 +02:00
vdeniaud force-pushed wip/78230-Nettoyage-de-quelques-settings-o from 8016152c1d to 99a83140b9 2023-06-07 11:58:07 +02:00 Compare
vdeniaud changed title from WIP: Nettoyage de quelques settings obsolètes (#78230) to Nettoyage de quelques settings obsolètes (#78230) 2023-06-07 11:59:22 +02:00
tnoel reviewed 2023-06-07 14:30:59 +02:00
src/authentic2/app_settings.py Outdated
@ -152,7 +152,6 @@ default_settings = dict(
A2_REGISTRATION_REALM=Setting(
default=None, definition='Default realm to assign to self-registrated users'
),
A2_REGISTRATION_GROUPS=Setting(default=(), definition='Default groups for self-registered users'),
tnoel commented 2023-06-07 14:30:47 +02:00
Owner
Copy Link

Pour le SHOW_LAST_CHAR ça ne pose pas de problème de le supprimer (ça va rien complétement casser) mais pour ce settings REGISTRATION_GROUPS on pourrait penser que des Authentic que nous ne suivons/gérons pas utilisent en fait cette affaire, et ça pourrait casser le fonctionnement...?

Pour le SHOW_LAST_CHAR ça ne pose pas de problème de le supprimer (ça va rien complétement casser) mais pour ce settings REGISTRATION_GROUPS on pourrait penser que des Authentic que nous ne suivons/gérons pas utilisent en fait cette affaire, et ça pourrait casser le fonctionnement...?
vdeniaud commented 2023-06-12 12:00:00 +02:00
Author
Owner
Copy Link

On est vraiment sur du très très vieux code qui date d'avant l'utilisation du modèle RBAC, on voit que le code ajoute des Group et pas des Role, c'est vraiment un retrait complètement safe.

On est vraiment sur du très très vieux code qui date d'avant l'utilisation du modèle RBAC, on voit que le code ajoute des Group et pas des Role, c'est vraiment un retrait complètement safe.
tnoel commented 2023-06-12 15:41:03 +02:00
Owner
Copy Link

Réponse acceptée !

Réponse acceptée !
tnoel marked this conversation as resolved
tnoel approved these changes 2023-06-12 15:42:05 +02:00
tnoel left a comment
Owner
Copy Link

De toute façon y'avait clairement pas de test, alors zou.

De toute façon y'avait clairement pas de test, alors zou.
vdeniaud commented 2023-06-12 15:53:10 +02:00
Author
Owner
Copy Link

Et zut/ouf mon cerveau reptilien s'est rappelé de la page d'authent GLC une demi-seconde avant de merger, https://moncompte.grandlyon.com/login/ et bingo ça utilise le script que je voulais supprimer. Il faut donc que je revois ma procédure pour scanner les settings.json d'authentic sur nos déploiements, j'étais tout à fait passé à côté... Et du JS pas touché depuis 5 ans qui marche encore, chapeau.

Et zut/ouf mon cerveau reptilien s'est rappelé de la page d'authent GLC une demi-seconde avant de merger, https://moncompte.grandlyon.com/login/ et bingo ça utilise le script que je voulais supprimer. Il faut donc que je revois ma procédure pour scanner les settings.json d'authentic sur nos déploiements, j'étais tout à fait passé à côté... Et du JS pas touché depuis 5 ans qui marche encore, chapeau.
vdeniaud force-pushed wip/78230-Nettoyage-de-quelques-settings-o from 99a83140b9 to 7823d8597b 2023-06-12 15:54:50 +02:00 Compare
vdeniaud merged commit cd04a94049 into main 2023-06-12 15:55:42 +02:00
vdeniaud deleted branch wip/78230-Nettoyage-de-quelques-settings-o 2023-06-12 15:55:42 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
tnoel
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/authentic#70
No description provided.
Delete Branch "wip/78230-Nettoyage-de-quelques-settings-o"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?