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 verification functions #40

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

davepgreene
Copy link

This PR adds two different implementations of a way to verify members of the webring.

The first, verify.mjs simply makes sure that the listed URL only returns a response in the 2xx range. This is a quick spot check to make sure that the site owner doesn't redirect to a different location. The second, verifyDNS.mjs is slightly more complicated. It verifies, using DNS TXT records, whether a site owner controls their own domain by requiring a TXT record on the listed hostname. Both of these functions use Netlify's scheduled functions feature. verify runs daily and verifyDNS runs weekly. Obviously that timing can be changed by updating the value in netlify.toml.

Neither of these implementations solves the question as to what to do if one of these functions actually encounters that scenario. I don't know much about Netlify or whether it has logging or alerting functionality. Hopefully this helps in the discussion in #33.

@ericwbailey ericwbailey mentioned this pull request Feb 6, 2023
1 task
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

Successfully merging this pull request may close these issues.

1 participant