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

Remove 2 second delay from first response and add IPv6 address support #597

Conversation

benagain
Copy link
Contributor

Fixes #554

Remove an approximate two second delay in response to the first request from a new socket connection, apparently only occuring on some Windows 10 machines when using .Net Core.

A side-effect of this fix is that is also allows connections to IPv6 addresses.

…st from a new socket connection, only occuring on some Windows 10 machines.

A side-effect of this fix is that is also allows connections to IPv6 addresses.
@sonarcloud
Copy link

sonarcloud bot commented Mar 21, 2021

@codecov
Copy link

codecov bot commented Mar 21, 2021

Codecov Report

Merging #597 (57e3ee3) into master (2fb0f92) will increase coverage by 0.03%.
The diff coverage is 66.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #597      +/-   ##
==========================================
+ Coverage   75.96%   76.00%   +0.03%     
==========================================
  Files         141      141              
  Lines        5463     5463              
  Branches      548      548              
==========================================
+ Hits         4150     4152       +2     
  Misses       1148     1148              
+ Partials      165      163       -2     
Impacted Files Coverage Δ
...ireMock.Net/Owin/AspNetCoreSelfHost.NETStandard.cs 55.31% <50.00%> (ø)
src/WireMock.Net/Owin/AspNetCoreSelfHost.cs 86.04% <100.00%> (ø)
src/WireMock.Net/Server/WireMockServer.Admin.cs 56.47% <0.00%> (+0.16%) ⬆️
src/WireMock.Net/Owin/Mappers/OwinRequestMapper.cs 97.95% <0.00%> (+2.04%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2fb0f92...57e3ee3. Read the comment docs.

@StefH StefH changed the title Remove 2 second delay from first response Remove 2 second delay from first response and add IPv6 address support Mar 21, 2021
@StefH
Copy link
Collaborator

StefH commented Mar 21, 2021

@benagain Thank you.

@StefH StefH merged commit 6f7d2c8 into WireMock-Net:master Mar 21, 2021
@benagain benagain deleted the remove-kestrel-first-response-delay-netcore branch March 22, 2021 04:50
@benagain
Copy link
Contributor Author

Thanks for merging so quickly @StefH! It's made a great improvent to our test runs 😄

benagain added a commit to SkillsFundingAgency/das-apprentice-commitments-web that referenced this pull request Mar 22, 2021
benagain added a commit to SkillsFundingAgency/das-apprentice-commitments-web that referenced this pull request Mar 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Requests are very slow (~2.1 sec) - bad performance
2 participants