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 support for selfhost.de #215

Closed
RMerl opened this issue Sep 25, 2018 · 11 comments
Closed

Add support for selfhost.de #215

RMerl opened this issue Sep 25, 2018 · 11 comments

Comments

@RMerl
Copy link
Contributor

RMerl commented Sep 25, 2018

Selfhost.de's DynDNS service uses a syntax compatible with inadyn's generic plugin. The following config file has been successfully tested with inadyn 2.4:

custom selfhost {
        ddns-server = carol.selfhost.de
        ddns-path = "/nic/update?hostname=%%h&myip=%%i"
        hostname = HOSTNAMEselfhost.eu
        username    = "userid"
        password    = "password"
        ddns-response = {"nochg", "good" }
}

The ddns-response line can be dropped after 2.4, now that plugins/generic.c can recognize the nochg response.

Authentication is done through Basic HTTP.

This could probably be easily be supported natively based on these info. I have a test account I'd be willing to privately share with the developer if needs be (that account was only used in my own work in adding support for it in my Asuswrt-Merlin project).

@troglobit
Copy link
Owner

Shouldn't be too difficult to fix. I'm a bit stressed out over other, more pressing, matters and projects atm., but anyone can help out. It looks very much like you can cut-and-paste a working plugin from just about any of the other ones.

@jorhett
Copy link
Contributor

jorhett commented Sep 27, 2018

@RMerl god knows you know how to create a PR right? ;-)

@RMerl
Copy link
Contributor Author

RMerl commented Sep 27, 2018

@jorhett While I've been doing dev work with Github for 6 years now, it was actually the first time I created a PR for another project.

Don't know if I'll have the time to work on a selfhost plugin myself, currently busy with my own project's release schedule.

@troglobit
Copy link
Owner

Hmm, they do look very similar to other Dyn.com clones. I can whip up speculative support on master if someone else can test it.

Do you know if they have a dedicated, or recommended, checkip-server?

troglobit added a commit that referenced this issue Sep 27, 2018
Signed-off-by: Joachim Nilsson <troglobit@gmail.com>
@troglobit
Copy link
Owner

If anyone could test e810cc3 I'd be very happy :)

@RMerl
Copy link
Contributor Author

RMerl commented Sep 27, 2018

I should be able to test it later today.

I can also email you my DDNS login credentials if you want to test it right now.

@RMerl
Copy link
Contributor Author

RMerl commented Sep 27, 2018

Can't find anything about a dedicated IP checker, but I did find this extended API documetnation:

https://selfhost.de/cgi-bin/selfhost?p=document&name=api

@troglobit
Copy link
Owner

@RMerl Thanks, much appreciated. If we can have this tested today I can do a new release tomorrow, or Saturday at the latest.

Yeah, I found that PDF too. Nothing much to add from what we (I) already know about how Dyn works,
so chances are good this will work. The currently hard coded checkip-server uses Dyn's checkip by default.

@RMerl
Copy link
Contributor Author

RMerl commented Sep 27, 2018

Login credentials emailed.

@RMerl
Copy link
Contributor Author

RMerl commented Sep 27, 2018

I was just about to send a pull request for the register/unregister - just finished testing it successfully on my end.

@troglobit
Copy link
Owner

@RMerl thank you for that, and sorry for missing the register/unregister stuff before. Having loging creds speeded up testing considerably! :)

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

No branches or pull requests

3 participants