Reactions getting stuck

[Yimura]

Please describe the problem you are having in as much detail as possible:
Reacting on a message made by my bot hangs. This hang gets resolved by pushing more reacts through the rest queue. I was unable to reproduce this problem but if contacted would be willing to give my code.

Include a reproducible code sample here, if possible:
Tried reproducing was unable to however my bot is a living example of this problem.
https://cdn.discordapp.com/attachments/334350562659139584/662982934097166336/music_-_Discord_2020-01-04_12-35-54.mp4
^^ Video where you can see the exact problem

Further details:

• discord.js version: v12
• Node.js version: v10.16.3 & v12.14.0
• Operating system: Debian 10.2 & Debian 10.1.0
• Priority this issue should have – please be realistic and elaborate if possible:
  Not very high as it happens in really rare cases and as how I was unable to reproduce this

Prior to v12 when my bot was still using v11.x this problem was not present

• I have also tested the issue on latest master, commit hash: d2ef029

[vladfrangu]

This is probably very easy to reproduce, looking at what your bot did before it "broke".

1. Queue up 5 reactions on a message
2. Create a new message and queue any amount of reactions
3. Delete the original message

My gut says that's a 100% reproducible issue. I also know why it's broken and I'll see about fixing soon!

[Yimura]

Alright I'm happy you know what is causing this.
I tried creating embeds and adding reactions to them but I didn't delete them in my try to reproduce this problem.

[Yimura]

Is there any progress on this issue?

[Yimura]

Managed to fix this by adding the following:

this.run();

above the following line in the file:
src/rest/RequestHandler.js#170

Before:

      try {
        const data = await parseResponse(res);
        if (res.status >= 400 && res.status < 500) {
          return reject(new DiscordAPIError(request.path, data, request.method, res.status)); // Before this line
        }
        return null;
      } catch (err) {
        return reject(
          new HTTPError(err.message, err.constructor.name, err.status, request.method, request.path)
        );
      }

After:

      try {
        const data = await parseResponse(res);
        if (res.status >= 400 && res.status < 500) {
          this.run(); // Added this
          return reject(new DiscordAPIError(request.path, data, request.method, res.status));
        }
        return null;
      } catch (err) {
        return reject(
          new HTTPError(err.message, err.constructor.name, err.status, request.method, request.path)
        );
      }

I'm using this as a hot fix as I'm assuming this is not the proper solution to this problem.

[Trezub]

Any update on this issue? I'm having this problem on 12.0.1

[Yimura]

As far as I know this hasn't been resolved, you can attempt to fix it yourself with the solution I gave above. The internals of the file don't seem to have changed since then.

Add below line 169 in file src/rest/RequestHandler.js

this.run();

[Trezub]

you can attempt to fix it yourself with the solution I gave above

This works! Thanks

Adding multiple reactions seems slow on this version, though

[Yimura]

you can attempt to fix it yourself with the solution I gave above

This works! Thanks

Adding multiple reactions seems slow on this version, though

Not sure, I think stricter timeouts have been added in discord.js or from Discord themselves.

[timotejroiko]

From what i understand, discord's API rate limits for reactions are 250ms, but discord.js has a rateLimitOffset which can be configured in the client options. The default value of the offset is 550ms, which effectively makes it wait at least 800ms before attempting to add the next reaction.

Reducing the offset in the client options will make the reactions faster, but you will need to add your own timings to them to avoid throwing rate limit errors.

Im sure this is something that could be improved in the future, but it would potentially require reworking a part of discord.js's internal rate limit prevention system