-
Notifications
You must be signed in to change notification settings - Fork 425
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
Feat/resolver #552
Feat/resolver #552
Conversation
fmt before pr agian. @vearne |
Codecov Report
@@ Coverage Diff @@
## master #552 +/- ##
==========================================
+ Coverage 24.21% 24.33% +0.12%
==========================================
Files 48 48
Lines 5010 5026 +16
==========================================
+ Hits 1213 1223 +10
- Misses 3598 3602 +4
- Partials 199 201 +2
Continue to review full report at Codecov.
|
ok
xujianhai666 <[email protected]> 于2020年11月26日周四 下午2:21写道:
… fmt before pr agian. @vearne <https://github.com/vearne>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#552 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AB6GYNDWZ5FBOPX4OPLEXSTSRXXW7ANCNFSM4TMFX72A>
.
|
sorry for that I merge incorrectly, could pull request again with format "https://github.com/apache/rocketmq/wiki/RIP-14-RocketMQ-Community-Operation-Conventions" |
Is something wrong in my comment? |
@xujianhai666 Is something wrong in my comment? |
What is the purpose of the change
In my scenario, my nameservers are hosts represented by a group of domain names. So I need a new resolver to resolve the nameservers.
Brief changelog
add dns Resolver to resolve name server
Verifying this change
go test . -v --run TestDNSResolver