Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Empêche un membre non staff d'en bannir un autre #4047

Merged
merged 1 commit into from
Dec 10, 2016
Merged

Empêche un membre non staff d'en bannir un autre #4047

merged 1 commit into from
Dec 10, 2016

Conversation

gllmc
Copy link
Member

@gllmc gllmc commented Dec 10, 2016

Q R
Type de modification correction d'une faille de sécurité

Avant tout, la faille est déjà fixée sur la prod, donc pas d'inquiétude !

Cette pull request corrige une faille de sécurité qui permettait à un membre de bannir, débannir ou annuler une sanction d'un autre membre sans avoir les droits staff. Un test est également ajouté pour éviter de reproduire cette erreur dans le futur.

Pour reproduire la faille (donc sur une version locale sans le fix), il faut envoyer une requête POST vers /membres/profil/modifier/{pk}/ avec pour paramètres :

  • ban-text : le motif du bannissement ;
  • csrfmiddlewaretoken : le token CSRF courant ;
  • ban : pour indiquer qu'il s'agit d'un bannissement.

QA

  • Vérifier qu'en suivant les instructions ci-dessus sans être staff, on reçoit bien une erreur 403 ;
  • Vérifier qu'un membre du staff peut toujours sanctionner un membre.

@coveralls
Copy link

coveralls commented Dec 10, 2016

Coverage Status

Coverage increased (+0.002%) to 87.684% when pulling aee6aa0 on GCodeur:correction_faille_ban into 6a1033b on zestedesavoir:prod.

@vhf vhf added C-Back Concerne le back-end Django S-BUG Corrige un problème labels Dec 10, 2016
@vhf
Copy link
Contributor

vhf commented Dec 10, 2016

Merci !

@vhf vhf merged commit 979e8ff into zestedesavoir:prod Dec 10, 2016
@vhf vhf modified the milestones: dev, Version de développement Dec 25, 2016
@gllmc gllmc deleted the correction_faille_ban branch July 17, 2017 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Back Concerne le back-end Django S-BUG Corrige un problème
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants