Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

fix: solver connection errors #427

Merged
merged 4 commits into from
Nov 18, 2024
Merged

fix: solver connection errors #427

merged 4 commits into from
Nov 18, 2024

Conversation

walkah
Copy link
Collaborator

@walkah walkah commented Nov 5, 2024 •
edited
Loading

Summary

This makes two changes:

  1. it uses KeyByRealIP for the rate limiter for use with load balancers, etc
  2. just logs low balance errors rather than triggering a restart to limit crash loops

@walkah walkah requested a review from a team as a code owner November 5, 2024 21:05
@cla-bot cla-bot bot added the cla-signed label Nov 5, 2024
@github-actions github-actions bot added the fix label Nov 5, 2024
@walkah walkah merged commit 52de132 into main Nov 18, 2024
4 checks passed
@walkah walkah deleted the walkah/fix-solver-connection-errors branch November 18, 2024 19:08
@lilypad-releases lilypad-releases bot mentioned this pull request Nov 18, 2024
Merged
walkah added a commit that referenced this pull request Dec 10, 2024
@walkah
fix: solver connection errors (#427)
164e996 
* fix: use KeyByRealIP for rate limiter

* fix: relax balance check errors
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@bgins bgins bgins approved these changes

Assignees

@walkah walkah

Labels
cla-signed fix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@walkah @bgins