entrouvert
/
authentic2-auth-fedict
16
0
Fork 0
Code Issues Pull Requests Releases Activity

do not rely on pkg_resources (#46476) #3

Merged
ecazenave merged 1 commits from wip/46476-no-pkgresources into main 2023-03-29 11:44:38 +02:00
Conversation 1 Commits 1 Files Changed 5 +12 -11
ecazenave commented 2023-03-21 15:25:30 +01:00
Owner
Copy Link
No description provided.
ecazenave added 1 commit 2023-03-21 15:25:31 +01:00
gitea/authentic2-auth-fedict/pipeline/head There was a failure building this commit Details
9bef11ef87 do not rely on pkg_resources (#46476)
ecazenave force-pushed wip/46476-no-pkgresources from 9bef11ef87 to abc4959962 2023-03-22 09:45:23 +01:00 Compare
ecazenave force-pushed wip/46476-no-pkgresources from abc4959962 to a32dc95770 2023-03-22 11:11:21 +01:00 Compare
ecazenave changed title from WIP: do not rely on pkg_resources (#46476) to do not rely on pkg_resources (#46476) 2023-03-22 11:23:16 +01:00
bdauvergne approved these changes 2023-03-23 11:00:14 +01:00
fpeters reviewed 2023-03-23 11:39:11 +01:00
fpeters left a comment
Owner
Copy Link

Résultat des courses on peut poser dès maintenant un :

if 'authentic2_auth_fedict' not in INSTALLED_APPS:
    INSTALLED_APPS += ('authentic2_auth_fedict',)
if 'authentic2_auth_fedict' not in TENANT_APPS:
    TENANT_APPS += ('authentic2_auth_fedict',)
if 'authentic2_auth_fedict.backends.FedictBackend' not in AUTHENTICATION_BACKENDS:
    AUTHENTICATION_BACKENDS += ('authentic2_auth_fedict.backends.FedictBackend',)

Quel problème il y aurait, dans ce patch, à ajoute un /etc/authentic2-multitenant/settings.d/00fedict.py qui poserait ça ?

> Résultat des courses on peut poser dès maintenant un : ``` if 'authentic2_auth_fedict' not in INSTALLED_APPS: INSTALLED_APPS += ('authentic2_auth_fedict',) if 'authentic2_auth_fedict' not in TENANT_APPS: TENANT_APPS += ('authentic2_auth_fedict',) if 'authentic2_auth_fedict.backends.FedictBackend' not in AUTHENTICATION_BACKENDS: AUTHENTICATION_BACKENDS += ('authentic2_auth_fedict.backends.FedictBackend',) ``` Quel problème il y aurait, dans ce patch, à ajoute un /etc/authentic2-multitenant/settings.d/00fedict.py qui poserait ça ?
ecazenave added 1 commit 2023-03-23 12:17:47 +01:00
gitea/authentic2-auth-fedict/pipeline/head This commit looks good Details
5ebef4c2f1 debian: provide a configuration file (#46476)
fpeters commented 2023-03-23 12:44:51 +01:00
Owner
Copy Link

Désolé, échec sur cette idée, après avoir vérifié il y a déjà un fichier local qui s'appelle 00fedict.py, qu'il ne faut pas effacer. Je vais donc plutôt voir avec imio pour assurer que ce fichier est modifié.

https://dev.entrouvert.org/issues/75741

Désolé, échec sur cette idée, après avoir vérifié il y a déjà un fichier local qui s'appelle 00fedict.py, qu'il ne faut pas effacer. Je vais donc plutôt voir avec imio pour assurer que ce fichier est modifié. https://dev.entrouvert.org/issues/75741
ecazenave force-pushed wip/46476-no-pkgresources from 5ebef4c2f1 to a32dc95770 2023-03-23 16:26:09 +01:00 Compare
ecazenave merged commit a32dc95770 into main 2023-03-29 11:44:38 +02:00
ecazenave deleted branch wip/46476-no-pkgresources 2023-03-29 11:44:38 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
bdauvergne
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/authentic2-auth-fedict#3
No description provided.
Delete Branch "wip/46476-no-pkgresources"

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?