Skip to content

Comments

feat(bindings/nodejs): add concurrent limit layer#6739

Merged
Xuanwo merged 6 commits intomainfrom
feat-concurrent-limit-layer
Oct 27, 2025
Merged

feat(bindings/nodejs): add concurrent limit layer#6739
Xuanwo merged 6 commits intomainfrom
feat-concurrent-limit-layer

Conversation

@kingsword09
Copy link
Contributor

Which issue does this PR close?

Rationale for this change

What changes are included in this PR?

  • Refactor layer code into separate module (src/layer.rs).
  • Add ConcurrentLimitLayer with permits and http_permits options.
  • Add Layer test.

Are there any user-facing changes?

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. releases-note/feat The PR implements a new feature or has a title that begins with "feat" labels Oct 24, 2025
@kingsword09 kingsword09 requested a review from Xuanwo as a code owner October 24, 2025 23:13
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Oct 24, 2025
Copy link
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Oct 27, 2025
@Xuanwo Xuanwo merged commit e8cbfbf into main Oct 27, 2025
70 checks passed
@Xuanwo Xuanwo deleted the feat-concurrent-limit-layer branch October 27, 2025 09:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer releases-note/feat The PR implements a new feature or has a title that begins with "feat" size:XL This PR changes 500-999 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants