-
-
Notifications
You must be signed in to change notification settings - Fork 189
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 NameSilo provider #866
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi there! Thanks for the PR! Nice code and thank you for the great documentation!
I have a few comments here and there, but overall it's pretty much already good code 😉 💯 !
Co-authored-by: Quentin McGaw <quentin.mcgaw@gmail.com>
- Splits create and update logic for simplicity. - Fixes scope of internal response struct. - Compresses example images to minimise repo size.
I have updated the PR based on the review suggestions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, thanks! 💯
NOTE: you may need to Can you add the following entry to .github/workflows/configs/mlc-config.json so the markdown links check ignores them: {
"pattern": "^https://www.namesilo.com/"
}, Given the markdown CI check errors:
Thanks!! |
Adds support for NameSilo. Resolves #313
I have tested locally and it seems to work for my domains. Wildcards and IPv6 are supported. The provider will create a new record if one does not already exist.
With credit to @Zeustopher for writing most of the readme in this comment.
The API documentation is here for reference: NameSilo API Reference.