Add Resolve-DnsName to QNAME Minimization under DNS validation section #1234

Merged
Strappazzon merged 4 commits from dig-qname into master 2019-09-06 00:54:39 +00:00
Strappazzon commented 2019-08-28 20:04:56 +00:00 (Migrated from github.com)

Description

dig on Windows doesn't exist so I added Resolve-DnsName.

Check List

## Description `dig` on Windows doesn't exist so I added [`Resolve-DnsName`](https://docs.microsoft.com/en-us/powershell/module/dnsclient/resolve-dnsname?view=win10-ps). #### Check List - [x] I have read and understand [the contributing guidelines](https://github.com/privacytoolsIO/privacytools.io/blob/master/.github/CONTRIBUTING.md). * Netlify preview for the mainly edited page: https://deploy-preview-1234--privacytools-io.netlify.com/providers/dns/
privacytoolsIO (Migrated from github.com) reviewed 2019-08-28 20:04:56 +00:00
netlify[bot] commented 2019-08-28 20:05:35 +00:00 (Migrated from github.com)

Deploy preview for privacytools-io ready!

Built with commit 5f2a2a176c

https://deploy-preview-1234--privacytools-io.netlify.com

Deploy preview for *privacytools-io* ready! Built with commit 5f2a2a176c7f2e3d5058fb2b2b7be0368cddab98 https://deploy-preview-1234--privacytools-io.netlify.com
Mikaela (Migrated from github.com) requested changes 2019-08-28 22:41:39 +00:00
Mikaela (Migrated from github.com) left a comment

Thanks, two comments however:

Thanks, two comments however: * I think the link makes it a bit more difficult to see that Resolve-DnsName is part of the command, but this isn't a blocker for me as it still has monospacing * Would you mind updating also the two other instances where `dig` is mentioned? * https://github.com/privacytoolsIO/privacytools.io/blob/master/.github/CONTRIBUTING.md#dns * https://github.com/privacytoolsIO/privacytools.io/blob/master/.github/ISSUE_TEMPLATE/7_DNS_provider.md (a HTML comment so invisible in the preview)
nitrohorse commented 2019-08-29 06:54:48 +00:00 (Migrated from github.com)

I think the link makes it a bit more difficult to see that Resolve-DnsName is part of the command, but this isn't a blocker for me as it still has monospacing...

Yeah, I wonder if we should just remove the hyperlinks from dig and Resolve-DnsName? I suspect users would research themselves more about these commands if they run into issues?

> I think the link makes it a bit more difficult to see that Resolve-DnsName is part of the command, but this isn't a blocker for me as it still has monospacing... Yeah, I wonder if we should just remove the hyperlinks from `dig` and `Resolve-DnsName`? I suspect users would research themselves more about these commands if they run into issues?
Strappazzon commented 2019-08-29 10:49:37 +00:00 (Migrated from github.com)

I made the requested changes.

Yeah, I wonder if we should just remove the hyperlinks from dig and Resolve-DnsName? I suspect users would research themselves more about these commands if they run into issues?

I agree, I removed both links.

> * Would you mind updating also the two other instances where `dig` is mentioned? > * https://github.com/privacytoolsIO/privacytools.io/blob/master/.github/CONTRIBUTING.md#dns > * https://github.com/privacytoolsIO/privacytools.io/blob/master/.github/ISSUE_TEMPLATE/7_DNS_provider.md (a HTML comment so invisible in the preview) I made the requested changes. > Yeah, I wonder if we should just remove the hyperlinks from `dig` and `Resolve-DnsName`? I suspect users would research themselves more about these commands if they run into issues? I agree, I removed both links.
Mikaela (Migrated from github.com) reviewed 2019-08-29 12:31:29 +00:00
Mikaela (Migrated from github.com) left a comment

Thanks :)

Thanks :)
nitrohorse (Migrated from github.com) reviewed 2019-08-29 14:50:40 +00:00
nitrohorse (Migrated from github.com) left a comment

LGTM 👍🏼

LGTM 👍🏼
nitrohorse (Migrated from github.com) approved these changes 2019-08-29 20:34:26 +00:00
Mikaela (Migrated from github.com) approved these changes 2019-08-30 06:39:11 +00:00
blacklight447 (Migrated from github.com) approved these changes 2019-09-01 11:55:33 +00:00
Mikaela commented 2019-09-01 20:47:44 +00:00 (Migrated from github.com)

@JonahAragon @BurungHantu1605 Any idea when you could check this? This is stuck on you due to codeowners.

@JonahAragon @BurungHantu1605 Any idea when you could check this? This is stuck on you due to codeowners.
jonah approved these changes 2019-09-05 22:39:55 +00:00
This repo is archived. You cannot comment on pull requests.
No Milestone
No Assignees
1 Participants
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: privacyguides/privacytools.io#1234
No description provided.