search

DraftBot-A-Discord-Adventure / DraftBot

A Discord adventure bot that speaks english and french
MIT License
225 stars 61 forks source link

[BUG]: Bonus ligue #2186

Closed As-Nodt closed 1 year ago

As-Nodt commented 1 year ago

Description du bug.

Lorsque j'ai voulu faire mon bonus ligue a 21h18 sur la shard 4, l'application ne répondait plus mais le jeu a tout de même validé une mission secondaire.

Selon vous, quelle est l'importance de ce bug ?

😐 Normale

Reproduction du bug.

Faire un bonus ligue.

Captures d'écran

Screenshot_20231008_211957

Sur quel shard avez vous rencontré le bug ?

Shard 4

voltou91 commented 1 year ago

Après avoir regardé les logs, ce n'est pas la commande bonus ligue le problème, c'est le bug du #2175 qui en est à l'origine. Capture d'écrans_20231009-001055~2.png

BastLast commented 1 year ago

du coup je ferme en duplicate

Procion06 commented 1 year ago

bonusligue

Shard 1 Aucun item reçu

Nargacaura commented 1 year ago

Le problème doit sûrement venir quelque part par là, visiblement, vu que l'XP est ajouté mais on n'a pas l'embed de réussite du leaguebonus: https://github.com/DraftBot-A-Discord-Adventure/DraftBot/blob/d1a40139493b70f0747ffa3aa3273cb2b5cb7ca1/src/commands/player/LeagueRewardCommand.ts#L99-L100

Procion06 commented 1 year ago

je ne sais pas si vous avez touché à quelque chose mais je suis tombé sur le rapport me disant que le bonusligue était dispo alors j'ai fait la commande pour tester et ça a marché ! BL

Nargacaura commented 1 year ago

T'as vraiment récupéré 4k XP, 4k d'argent et 8k points en 2 /leaguebonus le même jour? Ça sonne comme un exploit possible, ça, surtout si l'erreur s'est enchainée...

BastLast commented 1 year ago

@voltou91 si j'ai bien compris au final c'est pas le meme que #2175 parce que ça c'est du au defer reply ?

pwaillette commented 1 year ago

T'as vraiment récupéré 4k XP, 4k d'argent et 8k points en 2 /leaguebonus le même jour? Ça sonne comme un exploit possible, ça, surtout si l'erreur s'est enchainée...

On a pas eu de nouvelle par rapport à ça ?

BastLast commented 1 year ago

Jai pas le temps de m'en occuper faut que quelqu'un s'en charge

pwaillette commented 1 year ago

Jai pas le temps de m'en occuper faut que quelqu'un s'en charge

Si c'est simplement ajouter un deferReply je peux m'en charger je pense, mais avant de faire quoi que ce soit j'aimerais savoir si ce problème peut engendrer un exploit, sauf que j'ai ni les logs ni la réponse :/

BastLast commented 1 year ago

Jai pas le temps de m'en occuper faut que quelqu'un s'en charge

Si c'est simplement ajouter un deferReply je peux m'en charger je pense, mais avant de faire quoi que ce soit j'aimerais savoir si ce problème peut engendrer un exploit, sauf que j'ai ni les logs ni la réponse :/

Faut faire l'analyse, je sais que voltou a bossé dessus il sait mieux que moi ou ça en est

pwaillette commented 1 year ago

Jai pas le temps de m'en occuper faut que quelqu'un s'en charge

Si c'est simplement ajouter un deferReply je peux m'en charger je pense, mais avant de faire quoi que ce soit j'aimerais savoir si ce problème peut engendrer un exploit, sauf que j'ai ni les logs ni la réponse :/

Faut faire l'analyse, je sais que voltou a bosser dessus il sait mieux que moi ou ça en est

Je vais lui demander dans ce cas, de plus il me semble qu'il a accès aux logs en étant contributeur actif ce qui pourra aider

Procion06 commented 1 year ago

On a pas eu de nouvelle par rapport à ça ?

la question s'adresse à moi ?

BastLast commented 1 year ago

ok normalement j'ai fix le truc à confirmer avec @voltou91 :)

voltou91 commented 1 year ago

ok normalement j'ai fix le truc à confirmer avec @voltou91 :)

Euh à vérifier mais si tu deferReply c'est plus un interaction.Reply que tu met normalement, quasi sûr que ça va crash sauf si discord.js a géré le truc mais je doute vu que toute les autres librairies de discord c'est pareille. Si tu deferReply faut Followup