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

fix: avoid sending bot auth on token endpoints #7022

Merged

Conversation

SpaceEEC
Copy link
Member

Please describe the changes this PR makes and why it should be merged:

This PR changes requests that use a token to authorize themselves to not use bot auth (Authorization header).

See: discord/discord-api-docs#4142 (this might be fixed on Discord's end, but definitely can't hurt to be fixed here [too])

Status and versioning classification:

  • Code changes have been tested against the Discord API, or there are no code changes
  • I know how to update typings and have done so, or typings don't need updating

@iCrawl iCrawl added this to the Version 13.4 milestone Nov 22, 2021
@ImRodry
Copy link
Contributor

ImRodry commented Nov 22, 2021

Can this get a higher priority? My bot has been getting 1 hour rate limits after very few interactions on the edit reply endpoint and I believe this PR would fix it, or at least I hope it does

@iCrawl iCrawl merged commit 7efeff4 into discordjs:main Nov 23, 2021
@DV8FromTheWorld
Copy link

I recognize this has already been merged, however, it was stated by Discord that this is an internal bug in the platform, not something that libraries should need to handle.

Just wanted to make sure that got resurfaced here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

5 participants