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

trigger more name_server errors in more cases #1086

Merged
merged 1 commit into from Apr 28, 2020

Conversation

bluejekyll
Copy link
Member

@bluejekyll bluejekyll commented Apr 28, 2020

Based on some work from #1085 this hopefully addresses the issues seen in #933

@ktff and @gabelerner would be interesting if you could test with this PR with your environments.

@codecov
Copy link

codecov bot commented Apr 28, 2020

Codecov Report

Merging #1086 into master will increase coverage by 0.05%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master    #1086      +/-   ##
==========================================
+ Coverage   84.94%   84.99%   +0.05%     
==========================================
  Files         125      125              
  Lines       13407    13415       +8     
==========================================
+ Hits        11388    11402      +14     
+ Misses       2019     2013       -6     

@bluejekyll bluejekyll merged commit cb56f66 into master Apr 28, 2020
@bluejekyll bluejekyll deleted the fix-concurrent-resolution branch April 28, 2020 23:34
@ktff
Copy link

ktff commented Apr 29, 2020

Unfortunately we currently can't test it since Vector is still on v0.12 of trust-dns-resolver.

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

2 participants