Skip to content

Conversation

garrrikkotua
Copy link
Contributor

@garrrikkotua garrrikkotua commented Oct 19, 2023

Changes proposed ✍️

What

🤖 Generated by Copilot at cdd09e2

Increased the rate limit for Discord integration and optimized the Redis rate limiter. These changes improve the performance and reliability of the services that interact with external APIs.

🤖 Generated by Copilot at cdd09e2

Sing, O Muse, of the swift and skillful coder
Who tweaked the constants of the Discord bot
And matched the rate limit of the sender
To the mighty Discord's own allotted slot

Why

How

🤖 Generated by Copilot at cdd09e2

  • Increase the Discord API rate limit constant and comment the time unit (link)
  • Fix the Redis counter key initialization and expiration for the rate limiter (link, link)
  • Remove the redundant argument from the cache increment method in rateLimiter.ts (link)

Checklist ✅

  • Label appropriately with Feature, Improvement, or Bug.
  • Add screenshots to the PR description for relevant FE changes
  • New backend functionality has been unit-tested.
  • API documentation has been updated (if necessary) (see docs on API documentation).
  • Quality standards are met.

@garrrikkotua garrrikkotua merged commit fa2817f into main Oct 19, 2023
@garrrikkotua garrrikkotua deleted the fix/rate-limiter branch October 19, 2023 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant