Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

allow to limit client parallel requests #396

Merged
merged 4 commits into from
Sep 23, 2022

Conversation

jkralik
Copy link
Member

@jkralik jkralik commented Sep 21, 2022

No description provided.

@jkralik jkralik force-pushed the jkralik/feature/limitParallelRequests branch 5 times, most recently from f8b962a to 8a1fc9b Compare September 21, 2022 10:54
@codecov-commenter
Copy link

codecov-commenter commented Sep 21, 2022

Codecov Report

Merging #396 (d01db30) into v3 (ede835f) will decrease coverage by 0.01%.
The diff coverage is 77.50%.

@@            Coverage Diff             @@
##               v3     #396      +/-   ##
==========================================
- Coverage   70.33%   70.31%   -0.02%     
==========================================
  Files          67       68       +1     
  Lines        5110     5195      +85     
==========================================
+ Hits         3594     3653      +59     
- Misses       1129     1154      +25     
- Partials      387      388       +1     
Impacted Files Coverage Δ
mux/router.go 63.63% <ø> (ø)
options/config/common.go 100.00% <ø> (ø)
options/commonOptions.go 43.07% <38.18%> (-3.36%) ⬇️
dtls/server/config.go 64.28% <50.00%> (ø)
dtls/client.go 72.85% <57.14%> (ø)
tcp/server/config.go 61.11% <60.00%> (ø)
tcp/client.go 74.13% <66.66%> (ø)
udp/client.go 78.46% <66.66%> (ø)
tcp/client/session.go 74.19% <75.00%> (ø)
udp/server/config.go 71.42% <75.00%> (ø)
... and 20 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@jkralik jkralik force-pushed the jkralik/feature/limitParallelRequests branch 3 times, most recently from 0070660 to b26d820 Compare September 21, 2022 13:32
- exported: type name will be used as client.Conn by other packages, and that stutters; consider calling this Conn
- exported: type name will be used as inactivity.InactivityMonitor by other packages, and that stutters; consider calling this Monitor
@jkralik jkralik force-pushed the jkralik/feature/limitParallelRequests branch 5 times, most recently from d368df9 to 3742ae8 Compare September 22, 2022 10:57
@jkralik jkralik marked this pull request as ready for review September 22, 2022 11:00
@jkralik jkralik force-pushed the jkralik/feature/limitParallelRequests branch from d01db30 to 3e644d7 Compare September 23, 2022 13:17
@jkralik jkralik merged commit 2b0032d into v3 Sep 23, 2022
@jkralik jkralik deleted the jkralik/feature/limitParallelRequests branch September 23, 2022 13:25
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.

None yet

3 participants