-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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 DNS provider for Shellrent #2126
Conversation
It was not an easy job because only one page is in English and the response examples are not really JSON valid. |
It seems it's not recognizing my domain:
steps i did:
|
You should checkout my PR:
|
Thanks for the head up. dns request is passing thorugh now, I receive a missing domain zone error (although I have it up and running):
|
The problem is related to API response:
The doc is not accurate. The IDs are string, not int. |
I fixed the problem, can you try again? |
still receiving the same error:
Let me know if I can debug a bit more EDIT: just to confirm I deleted and re-git cloned the repo changing branch to feat/shellrent |
It's not exactly the same error: Before:
After:
I added debug logs, can you try again? |
here is the full new output:
i think i found the issue; i have two domains configured on my account and i'm trying to edit only one of them; it seems the reqeust is trying to change wrong domain, as you can see from the logs we might need to pass over the domain ID with username and api key as it was written on the wiki (although never imnplemented)? |
It's not what the log says. To find the domain ID, I should browse the domains.
As I already said: the Also, the implementation is not the same as mine. I'm trying to find the domain ID automatically. I added more logs and changed something, can you try again? |
here are new logs:
|
I found the problem, can you try again? |
new logs, seems you got progresses:
|
The error comes from the Shellrent API. |
I updated the code, can you try again? |
same logs at the end:
i can try to open them a ticket and see what they answer |
I think I found the problem, can you try again? |
latest log output:
|
I fixed the problem, can you try again? |
no errors, all should be fine now:
|
FYI most of the problems were related to the API documentation containing wrong information. I'm happy that is working now! I will clean "WIP" commits. Sponsoring is a good way to sustain open source maintainers: sponsor me |
fd5620e
to
f86c003
Compare
Thanks a lot for your patience and your support! |
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.
LGTM
Add DNS provider for Shellrent #2126 (comment)
go mod tidy
Ping @urgali, can you run the command (with your domain, email, and credentials)?
Closes #2124
Sponsoring is a good way to sustain open source maintainers: sponsor me