[DONE] Change is_staff to follow affiliation #972
Conversation
| @@ -130,6 +130,10 @@ def save(self, *args, **kwargs): | |||
| self.hashkey = hashlib.sha256( | |||
| (SECRET_KEY + self.user.username).encode("utf-8") | |||
| ).hexdigest() | |||
| if self.affiliation in AFFILIATION_STAFF : | |||
There was a problem hiding this comment.
Il y a déjà un user.is_staff = True dans la fonction pod/authentication/backend.py. Est-ce que ce n'est pas redondant du coup ?
There was a problem hiding this comment.
Pour Shibboleth et OIDC, oui, mais pour le CAS ? (et LDAP je crois que c'est bon aussi)
There was a problem hiding this comment.
Bonjour,
la modification du staff se fait pour le ldap : https://github.com/EsupPortail/Esup-Pod/blob/master/pod/authentication/populatedCASbackend.py#L240 mais egalement pour le CAS : https://github.com/EsupPortail/Esup-Pod/blob/master/pod/authentication/populatedCASbackend.py#L293
J'avoue ne pas comprendre le besoin, est-ce possible de m'expliquer ?
Merci
| @@ -130,6 +130,10 @@ def save(self, *args, **kwargs): | |||
| self.hashkey = hashlib.sha256( | |||
| (SECRET_KEY + self.user.username).encode("utf-8") | |||
| ).hexdigest() | |||
| if self.affiliation in AFFILIATION_STAFF : | |||
There was a problem hiding this comment.
Bonjour,
la modification du staff se fait pour le ldap : https://github.com/EsupPortail/Esup-Pod/blob/master/pod/authentication/populatedCASbackend.py#L240 mais egalement pour le CAS : https://github.com/EsupPortail/Esup-Pod/blob/master/pod/authentication/populatedCASbackend.py#L293
J'avoue ne pas comprendre le besoin, est-ce possible de m'expliquer ?
Merci
|
J'en ai parlé sur le rocketchat, en gros le champ is_staff ne se met pas à jour en fonction de l'affiliation (pourtant défini dans |
Bonjour Pierre et merci pour ton retour. Je vais regarder tout ca d'ici cette fin de semaine et reviens vite vers toi. |
|
au passage, la PR ne passe pas le test "flake8" car elle contient des espaces en fin de lignes. |
|
Vu avec Pierre en visio, le problème provenait de sa configuration du serveur SSO-CAS, son serveur ne remonte pas les affiliations de ses utilisateurs |
Follow is_staff dynamictly with AFFILIATION_STAFF and user affiliation.
It also revoke if user isn't part of affiliation staff anymore.