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

ldap: allow option to deactivate synchrozination for a realm (#6380) #26

Closed
smihai wants to merge 1 commits from wip/6380-ldap-add-option-to-deactivate-synchronization into main
pull from: wip/6380-ldap-add-option-to-deactivate-synchronization
merge into: entrouvert:main
Conversation 2 Commits 1 Files Changed 2 +29
smihai commented 2023-03-20 15:33:29 +01:00
Owner
Copy Link
No description provided.
fpeters requested changes 2023-03-20 15:49:41 +01:00
src/authentic2/backends/ldap_backend.py
@ -467,6 +467,7 @@ class LDAPBackend:
'lname_field': 'sn',
'timeout': 5,
'sync_timeout': 30,
'active_sync': True,
fpeters commented 2023-03-20 15:49:34 +01:00
Owner
Copy Link

Les paramètres qui contrôlent d'autres aspects s'appellent authentication/provisionning,

        'authentication': True,
        'provisionning': True,

Ici ça pourrait s'appeler synchronisation, pour rester sur la même forme ?

Les paramètres qui contrôlent d'autres aspects s'appellent authentication/provisionning, ``` 'authentication': True, 'provisionning': True, ``` Ici ça pourrait s'appeler synchronisation, pour rester sur la même forme ?
smihai commented 2023-03-20 15:56:15 +01:00
Author
Owner
Copy Link

Oui, tu as raison. Paramètre renommé.

Oui, tu as raison. Paramètre renommé.
smihai force-pushed wip/6380-ldap-add-option-to-deactivate-synchronization from 5df9bfac27 to b95e3aef75 2023-03-20 15:55:58 +01:00 Compare
smihai requested review from fpeters 2023-03-20 16:48:34 +01:00
smihai self-assigned this 2023-03-21 10:03:57 +01:00
pmarillonnet commented 2023-03-21 12:07:35 +01:00
Owner
Copy Link

À lire le code je ne comprends pas la différence entre cette nouvelle option et l’option provisionning déjà existante.

À lire le code je ne comprends pas la différence entre cette nouvelle option et l’option provisionning déjà existante.
smihai commented 2023-03-21 12:28:29 +01:00
Author
Owner
Copy Link

À lire le code je ne comprends pas la différence entre cette nouvelle option et l’option provisionning déjà existante.

Indeed, cette option fait déjà le job, depuis 9 mois: https://dev.entrouvert.org/issues/60492

> À lire le code je ne comprends pas la différence entre cette nouvelle option et l’option provisionning déjà existante. Indeed, cette option fait déjà le job, depuis 9 mois: https://dev.entrouvert.org/issues/60492
pmarillonnet closed this pull request 2023-03-21 15:58:37 +01:00
All checks were successful
gitea/authentic/pipeline/head This commit looks good
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
fpeters
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
smihai
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/authentic#26
No description provided.
Delete Branch "wip/6380-ldap-add-option-to-deactivate-synchronization"

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?