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

Support MX record pointing to root label, meaning no service available. #982

Merged
merged 1 commit into from Jan 4, 2020

Conversation

bluejekyll
Copy link
Member

fixes: #980

@bluejekyll bluejekyll changed the title reproduce error stack overflow from . as MX target Support MX record pointing to root label, meaning no service available. Jan 4, 2020
@bluejekyll
Copy link
Member Author

@Darkspirit this should fix the issue you discovered.

stop additional lookup on Root name
@codecov
Copy link

codecov bot commented Jan 4, 2020

Codecov Report

Merging #982 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #982      +/-   ##
==========================================
+ Coverage   85.87%   85.88%   +<.01%     
==========================================
  Files         122      122              
  Lines       13288    13289       +1     
==========================================
+ Hits        11411    11412       +1     
  Misses       1877     1877

@Darkspirit
Copy link
Contributor

Yes, it seems to be fixed. Thank you! :)

@bluejekyll bluejekyll merged commit ab19368 into master Jan 4, 2020
@bluejekyll bluejekyll deleted the add-support-for-no-service branch January 4, 2020 21:43
@bluejekyll
Copy link
Member Author

Ok, cool, thanks for finding this!

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.

thread 'trust-dns-server-runtime' has overflowed its stack
2 participants