Quel mot de passe faut-il taper ? [Réglé]
nic80 Membre non connecté
-
- Voir le profil du membre nic80
- Inscrit le : 06/08/2018
- Groupes :
-
Modérateur
Reprise du message précédent
Bonjour,J' ai fait quelques tests supplémentaires...
Parce que je comprenais pas "la combobox ne s'affiche que si au moins deux utilisateurs sont connectés", car l' agent polkit est lié à la session utilisateur en cours ( et théoriquement il ne peut pas y avoir deux utilisateurs différents connectés à la même session !)En fait la combobox n' apparait que s' il y a au moins deux utilisateurs (autres que root) dans le groupe wheel ( groupe qui est celui des administrateurs au sens de polkit). Dans ce cas, l' utilisateur root n' est d' ailleurs plus proposé.
Exemples ( usertest et nouvel_utilisateur sont dans le groupe wheel):
drakconf ( qui a besoin de droits administrateur):
drakrpm-update ( droits utilisateur par défaut).
Édité par nic80 Le 19/07/2020 à 16h35
Papoteur Membre non connecté
-
- Voir le profil du membre Papoteur
- Inscrit le : 03/10/2011
- Groupes :
-
Modérateur
-
Équipe Mageia
-
Administrateur
-
Forgeron
Peux-tu générer un patch et l'ajouter au rapport https://bugs.mageia.org/show_bug.cgi?id=15124 ?
Puis je demanderai à David de l'inclure dans Mageia 8, voir 7.
nic80 Membre non connecté
-
- Voir le profil du membre nic80
- Inscrit le : 06/08/2018
- Groupes :
-
Modérateur
Déjà fait
Papoteur Membre non connecté
-
- Voir le profil du membre Papoteur
- Inscrit le : 03/10/2011
- Groupes :
-
Modérateur
-
Équipe Mageia
-
Administrateur
-
Forgeron
Perso, je n'aurais pas fait la correction comme ça. J'aurais modifié le test et supprimé le cas "Password:" et son pendant.
Genre:
Code CPP :
if (requiresAdmin && !identity.isValid()) {
lblPassword->setText(i18n("Password for root:"));
} else {
lblPassword->setText(i18n("Password for %1:",
identity.toString().remove("unix-user:")));
}
nic80 Membre non connecté
-
- Voir le profil du membre nic80
- Inscrit le : 06/08/2018
- Groupes :
-
Modérateur
Non on est obligé d' avoir 3 cas à mon sens:
Si droits admin nécessaires
1) Il n' y a pas d' utilisateur dans le groupe wheel => mot de passe root
2) il y a un autre utilisateur que root dans le groupe wheel => mot de passe de celui ci en lieu et place de root ( il y a peu de chance que cela arrive, mais selon la loi de Murphy, si un utilisateur doit faire quelque chose qui n' est pas prévu, la probabilité qu' il la fasse sera égale à 1
) . En plus par défaut Polkit autorise ce cas ( voir le fichier /usr/share/polkit-1/rules.d/50-default.rules), il ne peut donc pas être supprimé. C' est le cas qui est traité dans le else du bloc "requires admin".)Si c' est juste pour confirmer que l' utilisateur courant
3) Alors on demande à l' utilisateur son mot de passe
Il y a un dernier cas, celui qui ne correspond à aucun des cas ci dessus, mais je n' ai pas compris à quoi il sert.
Édité par nic80 Le 19/07/2020 à 22h44
Papoteur Membre non connecté
-
- Voir le profil du membre Papoteur
- Inscrit le : 03/10/2011
- Groupes :
-
Modérateur
-
Équipe Mageia
-
Administrateur
-
Forgeron
A mon sens, c'est le nombre de types de sortie qui détermine le nombre de cas. Et ici, on ne veut plus que deux cas :
1- on veut le mot de passe de l'administrateur root, pour afficher "Password for root:"
2- on veut le mot de passe d'un autre utilisateur, pour afficher "Password for %1:"
Le cas 1 est déterminé par les conditions cumulatives :
requiresAdmin est vrai
identity.isValid() est faux
Le cas 2 est le cas complémentaire du 1. Il n'y a donc qu'un test à faire.
Papoteur Membre non connecté
-
- Voir le profil du membre Papoteur
- Inscrit le : 03/10/2011
- Groupes :
-
Modérateur
-
Équipe Mageia
-
Administrateur
-
Forgeron
Ton patch a été appliqué et validé dans Mageia. La question est donc maintenant réglée !
Répondre
Vous n'êtes pas autorisé à écrire dans cette catégorie