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

Add dyn53.io #820

Merged
merged 3 commits into from
May 24, 2019
Merged

Add dyn53.io #820

merged 3 commits into from
May 24, 2019

Conversation

schiederme
Copy link
Contributor

@schiederme schiederme commented May 23, 2019

  • Description of Organization
  • Reason for PSL Inclusion
  • DNS verification via dig
  • Run Syntax Checker (make test)

Description of Organization

Organization Website: https://port53.io

I'm the owner of port53, which will run a Anycast DNS Network.

Reason for PSL Inclusion

Because user.dyn53.io is used for customers, Cookie Security would be crucial.

DNS Verification via dig

dig +short TXT _psl.dyn53.io
"https://github.com/publicsuffix/list/pull/820"

make test

I could not getting make test work. It throws errors (make not list errors)

@schiederme
Copy link
Contributor Author

Maybe im just stupid. But I cannot find any infos regarding the make errors:
sh: 1: aclocal: not found autoreconf: aclocal failed with exit status: 127 Makefile:17: recipe for target 'libpsl-config' failed make: *** [libpsl-config] Error 127

@sleevi
Copy link
Contributor

sleevi commented May 24, 2019

Note: Your organization is also not presently sorted

@schiederme
Copy link
Contributor Author

@sleevi have seen it, working on a fix. Thank you for the information.

@schiederme
Copy link
Contributor Author

The position should now be fixed, can you review if this is right?

@sleevi sleevi added the h=sleevi (historical) Marked as approved and ready to merge by @sleevi label May 24, 2019
@sleevi sleevi self-requested a review May 24, 2019 21:46
@sleevi sleevi merged commit a1db0e8 into publicsuffix:master May 24, 2019
@jeffrey-pinyan-cleandns jeffrey-pinyan-cleandns mentioned this pull request Jul 6, 2023
10 tasks
jeffrey-pinyan-cleandns added a commit to jeffrey-pinyan-cleandns/publicsuffix-list that referenced this pull request Jul 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
h=sleevi (historical) Marked as approved and ready to merge by @sleevi
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants