Erreur 500 - réponse à un sujet

Hello tout le monde

En répondant à ce sujet : https://zestedesavoir.com/forums/sujet/8803/nouvelles-technos-web-pour-un-projet/

Je me suis retrouvé avec une erreur 500. D’après ma (rapide) recherche, c’est pas la première fois que ça arrive mais les bugs datent d’il y a 2 ans, alors j’ai supposé que c’en était un nouveau.

J’ai réessayé plusieurs fois.

La même chez moi.

Idem ! Par contre j’ai pu éditer mon premier message sans avoir d’erreur, si ça peut aider.

J’ai eu une double notification sur le message de Demandred, dont une persistente. Je suis presque certain que ça vous empêche d’y poster. J’ai vu un truc dans sentry aussi.

J’ai fixé ma notification, try again.

Toujours raté.

Bon ben d’accord, du coup va falloir attendre que quelqu’un intervienne, moi j’y peux pas grand chose.

Yop, pareil !

D’ailleurs je m’étais abonné au sujet, et visiblement y a des membres qui ont vainement tenté une petite dizaine de fois de poster dans le topic, puisque le notificateur de ZdS m’a spammé ma boîte mails

J’ai aussi la même notif persistante. Si tu sais la corriger, ça me rendrait heureux.

Je sais rien, navré.

J’ai fait disparaitre ma notification avec un clic sur le bouton "tout marquer comme lu" sur cette page : https://zestedesavoir.com/notifications/ .

Ah j’avais oublié ce bouton. Il marche pour moi aussi.

Thank

J’ai trouvé l’origine du bug mais pas la manière de le reproduire. Je vais essayer de nettoyer les notifications en double ce soir. En attendant, j’ai clos temporairement le sujet pour éviter des 500 pour rien.

https://github.com/zestedesavoir/zds-site/issues/4378

J’ai fait une PR un poil avant que gustavi ne crée le ticket.

Si ça peut aider hier j’ai essayé de répondre à ce sujet.

C’est débloqué. Je crois qu’on a trouvé pourquoi ça s’est produit, et comment éviter que ça se reproduise.

Sujet déverrouillé du coup.

Il y a eu un fix déployé ?

non, j’ai agis sur la prod.

C’est apparement une race condition sur un élément de la bdd qu’on n’a pas rendu atomique/transactionnel.

La Pr est en cours.

Nope. Mon hypothèse est que c’est une des méthodes de manager de ce genre : https://github.com/zestedesavoir/zds-site/blob/c597532273c241b7972a5075fab3e169ead94dfb/zds/notification/managers.py#L133-L158. C’est pas dans une transaction, du coup évidemment ça risque de foirer.

Là, le topic en question, c’est deux messages consécutifs à <2s d’interval (le 1er de Demandred, le 2nd d’Aloha). Je parie qu’ils sont partis sur 2 threads gunicorn et que quand un checkait si une sub existait, l’autre avait déjà checké mais pas encore écrit. Et paf.

Ce qui me fait penser que du coup, ça signifie que l’antispam n’est pas non plus atomique.

À une époque (bien lointaine maintenant) on avait fait une grande campagne de « transactionnalisation » de tout le code. Je suppose que ça vaudrait le coup d’en refaire une, et aussi de faire attention à ça dans les futures PR.

oui, il faudra.