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

Clarification of NextDNS logging policy #2434

Merged
merged 1 commit into from
Mar 13, 2024
Merged

Conversation

xe-3
Copy link
Contributor

@xe-3 xe-3 commented Mar 9, 2024

Changes proposed in this PR:

  • CONTEXT: This PR revises footnote 5 in the DNS section (NextDNS' logging policy) it builds upon and further clarifies 67614c3 and is an alternative to the proposed PR 2427.

  • PROBLEM: The current description needs clarification and seemingly contradicts itself (it begins by stating logging is opt-out and ends by saying no logs are kept without the user 'specifically requesting it'). It also does not differentiate NextDNS's public DNS servers dns.nextdns.io from the personal my.nextdns.io/<ID>.

  • There is a pending PR (2427) that proposes to amend that footnote, but the proposed changes in that PR leave out important information that can help readers make an informed choice. My assessment is that PR 2427 helpfully clarifies one inaccuracy but introduces its own inaccuracy by omission. The author of that PR has so far been unwilling to amend or modify it. So I am creating this PR as an alternative.

  • GOAL: The goal of this PR is to provide readers clearer and fuller information than the current footnote provides, and provide fuller information than the other proposed pull request (2427) and use neutral language.

  • SPECIFIC CHANGES: This PR is an improvement in three specific ways, (1) it clarifies the seemingly contradictory statements about logging (2) it explains to readers how the the logging policy will apply to NextDNS's public and personal DNS services, and (3) it alerts readers to the apparent contradiction between the NextDNS privacy policy, and the default account settings.

  • The proposed language is:

When used with a user account, NextDNS will enable insights and logging features by default (note: this seemingly conflicts with their privacy policy). You can choose retention time and log storage location for any logs you choose to keep, or disable logs altogether. If used without a user account, no data is logged. https://nextdns.io/privacy

  • IF this PR is accepted PR 2427 should be rejected. IF 2427 is accepted this PR should be rejected.

  • I have disclosed any relevant conflicts of interest in my post.
  • I agree to grant Privacy Guides a perpetual, worldwide, non-exclusive, transferable, royalty-free, irrevocable license with the right to sublicense such rights through multiple tiers of sublicensees, to reproduce, modify, display, perform, relicense, and distribute my contribution as part of this project.
  • I am the sole author of this work.
  • I agree to the Community Code of Conduct.

@xe-3 xe-3 changed the title Further clarification of NextDNS' logging policy in the DNS section Further clarification of NextDNS' logging policy in the DNS section t:correction Mar 9, 2024
@xe-3 xe-3 changed the title Further clarification of NextDNS' logging policy in the DNS section t:correction Further clarification of NextDNS' logging policy in the DNS section Mar 9, 2024
Copy link

netlify bot commented Mar 9, 2024

Deploy Preview for privacyguides ready!

Name Link
🔨 Latest commit bfa4b0f
🔍 Latest deploy log https://app.netlify.com/sites/privacyguides/deploys/65f136d8df7bd20008f931ce
😎 Deploy Preview https://deploy-preview-2434.preview.privacyguides.dev
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
4 paths audited
Performance: 78 (🟢 up 2 from production)
Accessibility: 91 (🔴 down 1 from production)
Best Practices: 98 (no change from production)
SEO: 90 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@dngray
Copy link
Member

dngray commented Mar 10, 2024

IF this PR is accepted PR 2427 should be rejected. IF 2427 is accepted this PR should be rejected.

I was looking at that and felt that it didn't really make things clearer.

logging features by default (note: this seemingly conflicts with their privacy policy).

I think we could make this sound less negative. The privacy policy relates to logging they would do for potentially other purposes like external reporting.

Obviously if you enable a logging feature because you want insight on your activity that has to be stored somewhere. I think it's unfair to say this is "contrary to privacy policy".

@archerallstars
Copy link

archerallstars commented Mar 10, 2024

This PR opens the footnote with:

When used with a user account,

I think it's worth mentioning that on https://nextdns.io, the "Try it now" buttons at both the top and the bottom of the page are described with "No signup required." With this, the users might assume that their account has not been created/used, since they didn't sign up for the service.

Moreover, there's no public DNS address (dns.nextdns.io) mentioning on the website at all. Even the mockup screenshot on the website that shows an example DNS address, is an address with the user account:

image

I doubt using the service without a user account is intended by the team at NextDNS at all. My point is, the service's main website doesn't tell the user how to use the service without a user account at all.

@dngray

logging features by default (note: this seemingly conflicts with their privacy policy).

I think we could make this sound less negative. The privacy policy relates to logging they would do for potentially other purposes like external reporting.

Obviously if you enable a logging feature because you want insight on your activity that has to be stored somewhere. I think it's unfair to say this is "contrary to privacy policy".

The problem with NextDNS privacy policy is not about the purpose of the log or the way the log is used, but the default behavior of the logging that's supposed to be opt-in, not opt-out. This is even more problematic considering that the logging is optional, regardless of whether the user has an account.

IMO, using the word "seemingly" is almost misleading, as the log can only be requested with the user account. To me, without any doubt, the service has violated its privacy policy.

@dngray dngray added c:providers service providers and similar centralized/federated services t:correction content corrections or errors labels Mar 10, 2024
@dngray
Copy link
Member

dngray commented Mar 10, 2024

"seemingly" is almost misleading

I don't particularly like that word either, it makes it sound as if this is the fault of Next DNS. To me it just seems to be whether or not the user wants to make use of a feature that requires the logging to operate.

I don't think this is the same as a opaque privacy policy where it might not be clear for whatever purpose.

For example with something like GDPR wouldn't this come under "legal basis" being that the feature needs it to work?

@xe-3
Copy link
Contributor Author

xe-3 commented Mar 10, 2024

I was looking at that and felt that it didn't really make things clearer.

Fair enough. Is there something specific that you feel is not clear? If it's just the note about the privacy policy, I would be willing to remove that.

The primary point I wanted to clarify was that:

  • With an account, logs are enabled by default (but can be disabled or configured to a user's preferences).
  • Without an account NextDNS keeps no logs.

I think we could make this sound less negative.

I agree. I struggled to find language that was both accurate, and neutral. I couldn't find a way to accomplish both so I chose accuracy. But I don't want to mislead readers into thinking logs are inherently bad, in the context of NextDNS logging is a feature not a flaw (but still a risk for some threat models)

How would you rephrase it? Or is your preference that the note on conflicting with the privacy policy be removed?

The privacy policy relates to logging they would do for potentially other purposes like external reporting.
Obviously if you enable a logging feature because you want insight on your activity that has to be stored somewhere. I think it's unfair to say this is "contrary to privacy policy"

On this point, I agree with @archerallstars:

The problem with NextDNS privacy policy is not about the purpose of the log or the way the log is used, but the default behavior of the logging that's supposed to be opt-in, not opt-out.

I suppose it isn't strictly necessary to explicitly note that conflict. As long as we make it clear that when you signup for an account logs are enabled by default (and I believe this PR does that, with or without the note on the privacy policy).

docs/dns.md Outdated
@@ -24,7 +24,7 @@ Encrypted DNS with third-party servers should only be used to get around basic [
[^2]: Cloudflare collects and stores only the limited DNS query data that is sent to the 1.1.1.1 resolver. The 1.1.1.1 resolver service does not log personal data, and the bulk of the limited non-personally identifiable query data is stored only for 25 hours. [https://developers.cloudflare.com/1.1.1.1/privacy/public-dns-resolver/](https://developers.cloudflare.com/1.1.1.1/privacy/public-dns-resolver/)
[^3]: Control D only logs for Premium resolvers with custom DNS profiles. Free resolvers do not log data. [https://controld.com/privacy](https://controld.com/privacy)
[^4]: Mullvad's DNS service is available to both subscribers and non-subscribers of Mullvad VPN. Their privacy policy explicitly claims they do not log DNS requests in any way. [https://mullvad.net/en/help/no-logging-data-policy/](https://mullvad.net/en/help/no-logging-data-policy/)
[^5]: NextDNS can provide insights and logging features on an opt-out basis. You can choose retention times and log storage locations for any logs you choose to keep. If it's not specifically requested, no data is logged. [https://nextdns.io/privacy](https://nextdns.io/privacy)
[^5]: When used with a user account, NextDNS will enable insights and logging features by default (note: this seemingly conflicts with their privacy policy). You can choose retention time and log storage location for any logs you choose to keep, or disable logs altogether. If used without a user account, no data is logged. [https://nextdns.io/privacy](https://nextdns.io/privacy)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
[^5]: When used with a user account, NextDNS will enable insights and logging features by default (note: this seemingly conflicts with their privacy policy). You can choose retention time and log storage location for any logs you choose to keep, or disable logs altogether. If used without a user account, no data is logged. [https://nextdns.io/privacy](https://nextdns.io/privacy)
[^5]: When used with a user account, NextDNS will enable insights and logging features by default as the feature requires it. You can choose retention time and log storage location for any logs you choose to keep, or disable logs altogether. If used without a user account, no data is logged. [https://nextdns.io/privacy](https://nextdns.io/privacy)

Copy link
Member

@dngray dngray Mar 10, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm thinking something like this, that clearly states the "logging/insights" feature require some "logging". Note: I have not used this specific feature so I am only guessing that's how it is.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dngray I'm onboard with your amendment. I'm editing it now.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, the PR has been updated. New language:

When used with a user account, NextDNS will enable insights and logging features by default (as some features require it). You can choose retention time and log storage location for any logs you choose to keep, or disable logs altogether. If used without a user account, no data is logged. https://nextdns.io/privacy

@archerallstars
Copy link

@dngray

For example with something like GDPR wouldn't this come under "legal basis" being that the feature needs it to work?

Even if the service needs the logging to work, I believe that the privacy policy should state it clearly. But currently, it's stated in the opposite direction.

Moreover, as a DNS resolver service, usage insight is not a hard requirement, e.g. resolving DNS queries. The logging is optional at best.

@archerallstars
Copy link

@xe-3

As long as we make it clear that when you signup for an account logs are enabled by default

Even without the user consent to sign up for an account, the logs are also enabled by default. Here is the button on the website:

image

It says: "No signup required."

@privacyguides-bot
Copy link
Collaborator

This pull request has been mentioned on Privacy Guides. There might be relevant details there:

https://discuss.privacyguides.net/t/nextdns-logging-is-opt-out-not-opt-in-as-stated-on-pgs-dns-resolvers-recommendations-page/17206/30

@xe-3
Copy link
Contributor Author

xe-3 commented Mar 10, 2024

The button in your screenshot is a ~1 week trial account (if you click the button, you'll see a big blue banner across the top that indicates that, and tells you you'll need to signup once trial period is over).

As to "no signup required" I agree with you. I may have accidentally used the term 'signup' in these comments somewhere, but I intentionally phrased the PR to avoid the term "signup" so as not to cause confusion.

The language I've used is:

"When used with a user account"

@dngray
Copy link
Member

dngray commented Mar 10, 2024

I think whats there now is good though.

Perhaps someone should raise with NextDNS the issue about the privacy policy not being entirely clear?

@xe-3
Copy link
Contributor Author

xe-3 commented Mar 10, 2024

Perhaps someone should raise with NextDNS the issue about the privacy policy not being entirely clear?

I've inquired about the privacy policy. I haven't heard back from them yet. When I do hear back, I'll post an update on the forum about it.

docs/dns.md Outdated Show resolved Hide resolved
@dngray dngray changed the title Further clarification of NextDNS' logging policy in the DNS section Clarification of NextDNS logging policy Mar 13, 2024
Co-authored-by: Jonah Aragon <jonah@triplebit.net>
Signed-off-by: Daniel Gray <dngray@privacyguides.org>
@dngray dngray merged commit bfa4b0f into privacyguides:main Mar 13, 2024
3 of 4 checks passed
@privacyguides-bot
Copy link
Collaborator

This pull request has been mentioned on Privacy Guides. There might be relevant details there:

https://discuss.privacyguides.net/t/nextdns-logging-is-opt-out-not-opt-in-as-stated-on-pgs-dns-resolvers-recommendations-page/17206/31

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c:providers service providers and similar centralized/federated services t:correction content corrections or errors
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants