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

Use .invalid domain name for InetAddress test #1179

Merged
merged 1 commit into from
Sep 18, 2020

Conversation

electrum
Copy link
Contributor

Some DNS setups such as ISP "DNS hijacking" will return an IP address for garbage names. Using the ".invalid" domain specified in RFC 6761 makes the test more reliable.

Some DNS setups such as ISP "DNS hijacking" will return an IP address
for garbage names. Using the ".invalid" domain specified in RFC 6761
makes the test more reliable.
@codecov-commenter
Copy link

codecov-commenter commented Sep 17, 2020

Codecov Report

Merging #1179 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1179   +/-   ##
=========================================
  Coverage     94.30%   94.30%           
  Complexity      455      455           
=========================================
  Files             2        2           
  Lines          6673     6673           
  Branches       1793     1793           
=========================================
  Hits           6293     6293           
  Misses          102      102           
  Partials        278      278           

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 52e8fd0...f2d205b. Read the comment docs.

@remkop remkop added this to the 4.6 milestone Sep 18, 2020
@remkop remkop merged commit b07a97d into remkop:master Sep 18, 2020
@remkop
Copy link
Owner

remkop commented Sep 18, 2020

Merged. Nice! I was not aware of RFC 6761, very good to know!

Thanks for the contribution!

remkop added a commit that referenced this pull request Sep 18, 2020
@electrum electrum deleted the inetaddress branch September 18, 2020 09:10
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.

None yet

3 participants