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.

Sign up for GitHub

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

Jump to bottom

erreur 500 lors d'une edition malencontreuse de commentaire saison 3 #3088

Closed
firm1 opened this issue Oct 14, 2015 · 9 comments
Closed

erreur 500 lors d'une edition malencontreuse de commentaire saison 3 #3088

firm1 opened this issue Oct 14, 2015 · 9 comments
Labels
C-Back Concerne le back-end Django S-BUG Corrige un problème
Milestone
Version 12.0

Comments

@firm1
Copy link
Contributor

firm1 commented Oct 14, 2015

Version beta-RC3

Url incriminée : http://beta.zestedesavoir.com/contenus/reactions/ajouter/?cite=76865&pk=83

Pouf erreur 500

On ne devrait jamais avoir d'erreur 500 dans le meilleur des mondes
@firm1 firm1 added S-Régression Corrige un problème sur un composant qui fonctionnait auparavant C-Back Concerne le back-end Django labels Oct 14, 2015
@GerardPaligot
Copy link
Member

De l'information complémentaire ? Les pks sont valides ? Le message est masqué ? ...

@firm1
Copy link
Contributor Author

firm1 commented Oct 14, 2015

J'ai pris une url au pif valide et j'ai changé la pk. Je ne sais pas si la pk existe quelque part.

@artragis
Copy link
Member

non en fait l'erreur a changé par rapport à avant (le bug "initial" est corrigé, mais deux corrections de bugs (citer un commentaire masquer) se sont fait de l'ombre.

l'erreur 500 arrive sur une pk qui n'existe pas en bdd, on devrait avoir une erreur 400 et non une 500. Je regarde ça ce midi.

@artragis
Copy link
Member

/!\ ce bug n'est pas propre à la béta, il est présent en prod.

@firm1
Copy link
Contributor Author

firm1 commented Oct 14, 2015

Avec quelle url tu arrive à le reproduire en prod ?

@GerardPaligot
Copy link
Member

Je n'arrive pas à reproduire en prod non plus.

@artragis
Copy link
Member

https://zestedesavoir.com/articles/message/nouveau/?article=321&cite=7776555

@GerardPaligot
Copy link
Member

Ok, pk était article. Je retag en conséquence.

@GerardPaligot GerardPaligot added S-BUG Corrige un problème and removed S-Régression Corrige un problème sur un composant qui fonctionnait auparavant labels Oct 14, 2015
@GerardPaligot GerardPaligot changed the title [beta 15.9] erreur 500 lors d'une edition malencontreuse de commentaire saison 3 erreur 500 lors d'une edition malencontreuse de commentaire saison 3 Oct 14, 2015
pierre-24 added a commit to pierre-24/zds-site that referenced this issue Oct 14, 2015
@pierre-24
Corrige une erreur 500 lors de la citation d'un commentaire inexistant (
789120b 
fix zestedesavoir#3088)

Fin de la saison 3.
@pierre-24 pierre-24 mentioned this issue Oct 14, 2015
Merged
@Eskimon Eskimon added this to the Version 15.9 milestone Oct 15, 2015
@Eskimon
Copy link
Contributor

Eskimon commented Oct 15, 2015

Ok

@Eskimon Eskimon closed this as completed Oct 15, 2015
pierre-24 added a commit to artragis/zds-site that referenced this issue Oct 16, 2015
@pierre-24 @artragis
Corrige une erreur 500 lors de la citation d'un commentaire inexistant (
15d646a 
fix zestedesavoir#3088)

Fin de la saison 3.
pierre-24 added a commit to artragis/zds-site that referenced this issue Oct 26, 2015
@pierre-24 @artragis
Corrige une erreur 500 lors de la citation d'un commentaire inexistant (
f537852 
fix zestedesavoir#3088)

Fin de la saison 3.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-Back Concerne le back-end Django S-BUG Corrige un problème
Projects
None yet
Milestone
Version 12.0
Development

No branches or pull requests
4 participants
@Eskimon @artragis @GerardPaligot @firm1