Cliquer sur l’enveloppe pour révéler un e-mail entrant devrait fonctionner pour le personnel et pour les utilisateurs qui consultent leurs propres publications. Cependant, en tant qu’utilisateur non-employé essayant de consulter sa propre publication, cela entraîne une erreur Javascript. L’utilisateur est capable de voir le JSON sous-jacent lorsque celui-ci est demandé manuellement.
Je peux confirmer que cela se produit également sur mon site de test pour les utilisateurs non-membres du personnel avec la même erreur : Uncaught Error: Failed to create an instance of 'controller:raw-email'. Most likely an improperly defined class or an invalid module export.
Cela a été signalé pour une attention particulière.
Ça a l’air légitime ! Merci d’avoir soulevé le problème, étant donné que le problème est si mineur, je vais y mettre un pr-welcome. Je soutiens la correction super triviale qui consiste simplement à ne rien appeler ici, je suis indécis quant à l’exposition complète des informations d’e-mail à soi-même pour les non-modérateurs.
pour ne rien appeler ; Je l’ai utilisé pour déboguer des problèmes d’e-mail d’innombrables fois en tant qu’administrateur, je n’ai jamais regardé mes propres messages en tant que modérateur ou utilisateur régulier, nulle part.
Je vois que cette pull request a été fermée (silencieusement), je suppose parce que j’ai honnêtement du mal à voir comment tester une fonction d’une seule ligne qui ne contient que is_staff?.
Veuillez me faire savoir si cela sera fusionné à l’avenir, car sinon je devrai créer un plugin pour résoudre ce problème de RGPD…
Je peux plus ou moins accepter de ne pas avoir de test côté client, mais côté API, cela devrait être trivial à tester. Vous créez un objet de publication, un objet utilisateur, vérifiez que l’utilisateur ne peut pas afficher l’e-mail brut lorsque la publication est supprimée dans les tests du gardien.
J’essaierai de comprendre pourquoi la PR a été fermée.
C’est un changement de comportement cependant, je suis curieux de savoir pourquoi nous optons pour la suppression de la fonctionnalité plutôt que pour la correction du bug.
C’est certainement étrange car je ne pense pas que cela était prévu et ce n’est pas moi qui l’ai fermée. Du moins pas intentionnellement et pas à ma connaissance. Quoi qu’il en soit, je ferai le suivi de la PR sous peu.
Quelque chose d’étrange dans l’état de la PR. Bug Github peut-être ?
La clôture était en effet très étrange, car il n’y avait pas d’utilisateur actif et je n’ai reçu aucune notification.
Heureux d’apprendre que ce n’était pas intentionnel !