Skip to content

Issue #35 limiter is not working - sample with promise - #70

Closed
boly38 wants to merge 1 commit into
jhurliman:mainfrom
boly38:35-limiter-not-working
Closed

boly38 wants to merge 1 commit into
jhurliman:mainfrom
boly38:35-limiter-not-working

Conversation

@boly38

@boly38 boly38 commented Apr 4, 2021

Copy link
Copy Markdown

add sample with promise to discuss about #35

@boly38 boly38 mentioned this pull request Apr 4, 2021
@jhurliman

Copy link
Copy Markdown
Owner

Thanks @boly38 for the Promise example and investigation of #35. The library now has a native Promise API and options-object constructor; the callback wrapper and positional constructor in this old sample no longer apply. #105 includes runnable await-based pacing examples and a deterministic regression for the reported 5,000-request scenario. Closing this obsolete example PR with credit here; the non-atomic multi-limiter composition caveat remains documented in #35.

@jhurliman jhurliman closed this Sep 10, 2026
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.

2 participants