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

In the UI warn the user if they are not using well behaved TLD #440

Open
dpc opened this issue Jun 25, 2024 · 5 comments
Open

In the UI warn the user if they are not using well behaved TLD #440

dpc opened this issue Jun 25, 2024 · 5 comments

Comments

@dpc
Copy link
Contributor

dpc commented Jun 25, 2024

When the guardian pastes in their domain that is not in org, com, info and possibly some other reputable TLDs, display warning message that some TLDs are know to ban domains for no good reason, and it will render affected peer unusable.

@elsirion
Copy link
Contributor

Especially TLDs held by the same organization as .xyz should be blacklisted.

@Kodylow
Copy link
Member

Kodylow commented Aug 10, 2024

Think this is obviated by the API rotation PR and since we're pushing that as the recommended action if their TLD messes with them. Closing

@Kodylow Kodylow closed this as completed Aug 10, 2024
@elsirion
Copy link
Contributor

Rotation is still a pita, especially if the user is on a 3rd party hoster. I think it's still worth it.

@elsirion elsirion reopened this Aug 10, 2024
@Kodylow
Copy link
Member

Kodylow commented Aug 10, 2024

What specifically should we warn here and where should I put the warning? It's an env config, should I pop up the error every time they login or just like hide the warning in danger zone? not sure there's any good place to put this.

Maybe a warning only in the setup steps like just show a banner over the RoleSelector saying that their dns sucks?

@elsirion
Copy link
Contributor

Maybe a warning only in the setup steps like just show a banner over the RoleSelector saying that their dns sucks?

Yep, that's what I thought.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants