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

fix: Update last criterion on MFA tools page #2618

Merged
merged 1 commit into from
Jun 26, 2024

Conversation

redoomed1
Copy link
Contributor

@redoomed1 redoomed1 commented Jun 17, 2024

Changes proposed in this PR:

  • I agree to the terms listed below:
    Contribution terms (click to expand) 1) I am the sole author of this work. 2) 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. 3) I have disclosed any relevant conflicts of interest in my post. 4) I agree to the Community Code of Conduct.

Copy link

github-actions bot commented Jun 17, 2024

Your preview is ready!

Name Link
🔨 Latest commit 1eb410c
😎 Preview https://2618--glowing-salamander-8d7127.netlify.app/

@redoomed1 redoomed1 changed the title Update last criterion on MFA tools page fix: Update last criterion on MFA tools page Jun 17, 2024
@jonaharagon
Copy link
Member

Hm, this is an interesting point. I think that @rollsicecream was correct in #2608 (comment) that this criteria doesn't affect Ente Auth, but I can imagine that "third-party" is not very clear. The changes here sound easier to understand 👍

Copy link
Member

@jonaharagon jonaharagon left a comment

Choose a reason for hiding this comment

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

What about this?

docs/multi-factor-authentication.md Outdated Show resolved Hide resolved
@redoomed1
Copy link
Contributor Author

redoomed1 commented Jun 18, 2024

I agree that the "third-party" wording is not very clear: is the first party the service provider (Ente Auth, in this case) or the operating system?

I like your suggested change because it simplifies the criterion by removing this question from consideration.

@freddy-m freddy-m requested a review from dngray June 23, 2024 17:55
@freddy-m freddy-m added the c:providers service providers and similar centralized/federated services label Jun 23, 2024
- Update links for Qubes VPN docs
- Update Nitrokey internal link

Signed-off-by: Jonah Aragon <jonah@privacyguides.org>
Signed-off-by: Freddy <freddy@privacyguides.org>
Signed-off-by: Daniel Gray <dngray@privacyguides.org>
@dngray dngray merged commit 9083180 into privacyguides:main Jun 26, 2024
6 checks passed
@redoomed1 redoomed1 deleted the update-mfa-criterion branch June 26, 2024 04:16
@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/2024-07-15/19478/1

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
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Regarding the last criterion on the MFA tools page
5 participants