Skip to content

api: implement better rate limiting backoff#224

Merged
mattklein123 merged 2 commits intomainfrom
status
Sep 2, 2025
Merged

api: implement better rate limiting backoff#224
mattklein123 merged 2 commits intomainfrom
status

Conversation

@mattklein123
Copy link
Contributor

  1. Remove shutdown handling in the API. It makes the code convoluted and
    doesn't do anything useful. Instead just move the shutdown wait to
    the top level future.
  2. Add retry after handling in the API.
  3. Move gRPC Code to the codec crate so it can be used in client code.
  4. Some cleanups in order to remove a bunch of clones during config
    processing.

Fixes BIT-5835

@mattklein123
Copy link
Contributor Author

mattklein123 commented Aug 31, 2025

@mattklein123 mattklein123 force-pushed the status branch 2 times, most recently from 1a78026 to b170084 Compare August 31, 2025 15:37
1) Remove shutdown handling in the API. It makes the code convoluted and
   doesn't do anything useful. Instead just move the shutdown wait to
   the top level future.
2) Add retry after handling in the API.
3) Move gRPC Code to the codec crate so it can be used in client code.
4) Some cleanups in order to remove a bunch of clones during config
   processing.

Fixes BIT-5835

Signed-off-by: Matt Klein <mklein@bitdrift.io>
Signed-off-by: Matt Klein <mklein@bitdrift.io>
@mattklein123 mattklein123 enabled auto-merge (squash) September 2, 2025 15:45
@mattklein123 mattklein123 merged commit 4d4873c into main Sep 2, 2025
5 checks passed
@mattklein123 mattklein123 deleted the status branch September 2, 2025 15:48
@github-actions github-actions bot locked and limited conversation to collaborators Sep 2, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants