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

[beta] La recherche permet de lire du contenu auquel on ne doit pas avoir accès #2874

Closed
firm1 opened this issue Jul 1, 2015 · 10 comments
Closed
Labels
C-Back Concerne le back-end Django S-Régression Corrige un problème sur un composant qui fonctionnait auparavant
Milestone

Comments

@firm1
Copy link
Contributor

firm1 commented Jul 1, 2015

Prérequis.

Appartenir à un groupe qui n'a aucun privilège particulier (dev par exemple).

Url incriminée : http://beta.zestedesavoir.com/rechercher/?q=hello+staff

On voit dans la liste des résultats des informations censés être réservés au forum staff. Ce qui n'est pas normal, d'autant plus quand essayant d'y acceder on tombe bien sur une 403. Le fait est qu'on peut ainsi lire des bout de conversations qui ne nous regarde guère.

Il s'agit donc ici d'une regression. j'hésite à tagguer en bloquant car on se rapproche de la faille ici.

@firm1 firm1 added S-Régression Corrige un problème sur un composant qui fonctionnait auparavant C-Back Concerne le back-end Django labels Jul 1, 2015
@DevHugo
Copy link
Contributor

DevHugo commented Jul 1, 2015

Lesquels ? si je suis ton lien, on peut accéder au sujet sans se taper de 403 en tant que membres. Tu as pas un lien qui marche pas car j'arrive pas à reproduire en local.

Ton user à plusieurs groupes ? ou pas ? Quels droits sur le sujet ?

@firm1
Copy link
Contributor Author

firm1 commented Jul 1, 2015

Je t'ai rajouté au groupe devs sur le beta pour que tu puisse vérifier. C'est le prérequis pour reproduire la regression

@DevHugo
Copy link
Contributor

DevHugo commented Jul 1, 2015

Ok, je vois le bug, c'est des groupes créé avec l'interface admin de Django ?

@firm1
Copy link
Contributor Author

firm1 commented Jul 1, 2015

c'est des groupes créé avec l'interface admin de Django ?

Oui

@DevHugo
Copy link
Contributor

DevHugo commented Jul 1, 2015

C'est surement cette ligne mais avec le bug haystack, c'est vite compliqué: https://github.com/zestedesavoir/zds-site/blob/release-v15.6/zds/search/views.py#L44

Je vais finalement pencher pour la solution, valeur sentinelle.

@SpaceFox
Copy link
Contributor

SpaceFox commented Jul 3, 2015

Et si on donne un privilège bidon à tous les groupes (exemple : le droit de lire les forums publics), est-ce que ça ne contourne pas le problème ?

@SpaceFox
Copy link
Contributor

SpaceFox commented Jul 3, 2015

Parce que ça me ferait chier de bloquer encore une release pour un problème qui ne concerne à priori que quelques personnes.

@Eskimon
Copy link
Contributor

Eskimon commented Jul 3, 2015

Au contraire, la a priori seul les dev peuvent voir des extraits des fofos staff ou CA alors qu'ils ne devraient pas (et les staff peuvent voir le fofo CA)

@DevHugo
Copy link
Contributor

DevHugo commented Jul 3, 2015

Voila, c'est corrigé comme ça, on en parle plus.

@SpaceFox
Copy link
Contributor

SpaceFox commented Jul 4, 2015

Corrigé !

@SpaceFox SpaceFox closed this as completed Jul 4, 2015
@SpaceFox SpaceFox added this to the Version 15.6 milestone Jul 4, 2015
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-Régression Corrige un problème sur un composant qui fonctionnait auparavant
Projects
None yet
Development

No branches or pull requests

4 participants