entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 40 Releases Activity

doublon d'utilisateur au provisionning (#75777) #192

Open
bdauvergne wants to merge 2 commits from wip/75777-Doublon-d-utilisateur-au-provisi into main
pull from: wip/75777-Doublon-d-utilisateur-au-provisi
merge into: entrouvert:main
Conversation 1 Commits 2 Files Changed 4 +148 -62
bdauvergne commented 2023-03-24 12:03:33 +01:00
Owner
Copy Link
No description provided.
bdauvergne added 1 commit 2023-03-24 12:03:34 +01:00
bdauvergne changed title from WIP: doublone d'utilisateur au provisionning (#75777) to WIP: doublon d'utilisateur au provisionning (#75777) 2023-03-24 12:03:51 +01:00
bdauvergne force-pushed wip/75777-Doublon-d-utilisateur-au-provisi from 2d86c28ea9 to 9919256ee3 2023-03-24 22:48:01 +01:00 Compare
bdauvergne changed title from WIP: doublon d'utilisateur au provisionning (#75777) to doublon d'utilisateur au provisionning (#75777) 2023-03-24 22:48:12 +01:00
fpeters requested changes 2023-03-25 14:28:50 +01:00
Dismissed
fpeters left a comment
Owner
Copy Link

(jenkins)

(jenkins)
bdauvergne force-pushed wip/75777-Doublon-d-utilisateur-au-provisi from 9919256ee3 to 2fc4d15094 2023-03-25 15:26:08 +01:00 Compare
bdauvergne force-pushed wip/75777-Doublon-d-utilisateur-au-provisi from 2fc4d15094 to b000de2f9b 2023-03-25 16:21:40 +01:00 Compare
bdauvergne requested review from fpeters 2023-03-25 16:41:01 +01:00
fpeters refused to review 2023-03-25 20:57:22 +01:00
fpeters dismissed fpeters’s review 2023-03-25 20:58:15 +01:00
bdauvergne force-pushed wip/75777-Doublon-d-utilisateur-au-provisi from b000de2f9b to b0feb59521 2024-01-10 17:14:10 +01:00 Compare
bdauvergne commented 2024-01-10 17:14:28 +01:00
Author
Owner
Copy Link

Rebasé sur la transformation de hobo_notify en commande django.

Rebasé sur la transformation de hobo_notify en commande django.
bdauvergne force-pushed wip/75777-Doublon-d-utilisateur-au-provisi from b0feb59521 to 3cfebfda8b 2024-01-17 09:52:46 +01:00 Compare
bdauvergne force-pushed wip/75777-Doublon-d-utilisateur-au-provisi from 3cfebfda8b to 0bc7330bcd 2024-01-17 09:55:25 +01:00 Compare
smihai reviewed 2024-01-17 10:31:30 +01:00
wcs/qommon/saml2.py Outdated
@ -539,6 +540,35 @@ class Saml2Directory(Directory):
if save:
user.store()
def get_or_create_user_by_name_id(self, login, ni, retry=0):
smihai commented 2024-01-17 10:31:22 +01:00
Owner
Copy Link

T'ajoutes cette méthode uniquement pour les tests ? Elle ne devrait pas être utilisée lors du provisionning ?

T'ajoutes cette méthode uniquement pour les tests ? Elle ne devrait pas être utilisée lors du provisionning ?
bdauvergne commented 2024-01-27 15:22:06 +01:00
Author
Owner
Copy Link

C'est le souci avec les relectures qui prennent 10 mois c'est que je ne sais plus mais à regarder oui j'ai séparé le code pour le rendre plus facilement testable en isolation, soit j'ai oublié de le rebrancher dans lookup_user() initialement soit ça s'est perdu dans un rebase, je ne peux plus voir les versions intermédiaires du patch, gitea ne les a visiblement pas conservées. En attendant j'ai branché get_user_or_create_user_by_name_id dans lookup_user dans cette dernière version.

C'est le souci avec les relectures qui prennent 10 mois c'est que je ne sais plus mais à regarder oui j'ai séparé le code pour le rendre plus facilement testable en isolation, soit j'ai oublié de le rebrancher dans lookup_user() initialement soit ça s'est perdu dans un rebase, je ne peux plus voir les versions intermédiaires du patch, gitea ne les a visiblement pas conservées. En attendant j'ai branché get_user_or_create_user_by_name_id dans lookup_user dans cette dernière version.
fpeters requested changes 2024-01-26 13:50:33 +01:00
Dismissed
fpeters left a comment
Owner
Copy Link

(commentaire de Serghei)

(commentaire de Serghei)
fpeters dismissed fpeters’s review 2024-01-26 13:50:47 +01:00
Reason:

(je passais juste le message)

bdauvergne force-pushed wip/75777-Doublon-d-utilisateur-au-provisi from 0bc7330bcd to 2b97ab8083 2024-01-27 15:17:33 +01:00 Compare
bdauvergne force-pushed wip/75777-Doublon-d-utilisateur-au-provisi from 2b97ab8083 to 1cd07f836c 2024-01-27 15:28:16 +01:00 Compare
bdauvergne requested review from smihai 2024-01-27 15:36:06 +01:00
All checks were successful
gitea/wcs/pipeline/head This commit looks good
Details
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
Sign in to join this conversation.
Reviewers
No reviewers
smihai
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 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/wcs#192
No description provided.
Delete Branch "wip/75777-Doublon-d-utilisateur-au-provisi"

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?