doc: note about dns ANY queries / RFC 8482#26695
Uh oh!
There was an error while loading. Please reload this page.
Conversation
nodejs-github-bot added dns 
doc/api/dns.md Outdated
There was a problem hiding this comment.
Can you make these into links to the other functions in this page?
There was a problem hiding this comment.
Would the text/markdown below work for you instead? I'm trying to avoid unnecessarily-lengthy sentences and unneeded details. I'm also enforcing some style rules here (like avoid personal pronouns).
Note that the operator of an authoritative DNS server might choose not to respond to `ANY` queries. It may be better to call individual methods like `resolve4()`, `resolveMx()`, and so on. For more information, see [RFC 8482][].I'd also be OK with upgrading "It may be better" to "It is almost always better" or "It is always better" or "It is better", or "Whenever possible, use..." I went with "It may be better" because it seemed to most closely reflect your original text ("you may want to"). But if a stronger statement is correct and appropriate, let's use it.
Trott commented Mar 16, 2019
(By the way, the style guide is at https://github.com/nodejs/node/blob/master/doc/STYLE_GUIDE.md. Worth looking at if you plan on writing lots of docs here, but less so if you plan on dropping a little piece in here or there and don't mind someone like me nit-picking the crud out of it. 😆 ) |
tlhunter commented Mar 16, 2019
I modified the text to be more concise and use reference-style links. Note that I'm using the |
Trott commented Mar 16, 2019
BridgeAR added the author ready 
Uh oh!
There was an error while loading. Please reload this page.
Co-Authored-By: tlhunter <[email protected]>
vsemozhetbyt commented Mar 17, 2019
PR-URL: #26695 Reviewed-By: Gus Caplan <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Bryan English <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
vsemozhetbyt commented Mar 17, 2019
Landed in 3b6344c (with a nit fix in bottom references sorting). |
richardlau commented Mar 18, 2019
We can normalize on |
PR-URL: nodejs#26695 Reviewed-By: Gus Caplan <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Bryan English <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
PR-URL: #26695 Reviewed-By: Gus Caplan <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Bryan English <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
PR-URL: #26695 Reviewed-By: Gus Caplan <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Bryan English <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
10 participants
This was inspired by the Cloudflare post, RFC8482 - Saying goodbye to ANY / RFC 8482.
The post calls out that the
ANYquery isn't always honored by various services and so this PR warns the developer that such discrepancies can exist and that they may need to call other methods one at a time instead.Checklist