ddns-path length?

[vdubikov]

Hi everyone!

Trying to write config to update Beget DNS with Beget.API, and what i've got:
ddns-path = '/api/dns/changeRecords?login=%u&passwd=%p&input_format=json&output_format=json&input_data={"fqdn":"%h","records":{"A":[{"priority":10,"value":"%i"}]}}'
works as it should,
but
ddns-path = '/api/dns/changeRecords?login=%u&passwd=%p&input_format=json&output_format=json&input_data={"fqdn":"%h","records":{"A":[{"priority":10,"value":"%i"}],"MX":[{"priority":10,"value":"mx1.beget.com"},{"priority":20,"value":"mx2.beget.com"}],"TXT":[{"priority":10,"value":"v=spf1 redirect=beget.com"}]}}'
for some reasons gives error with ifconfig.me

inadyn[480347]: Communication with checkip server ifconfig.me failed, run again with 'inadyn -l debug' if problem persists
inadyn[480347]: Failed to get IP address for custom, giving up!

Tried to determine max length of ddns-path by shortening value and came to 256... Of cource it still not work with Beget.API as i want it, but error with ifconfig.me is gone.

custom section of config

custom host.name {
	checkip-server = ifconfig.me
	checkip-path = /ip
	checkip-ssl = false
	username = user
	password = password
	ddns-server = api.beget.com
	ddns-path = '/api/dns/changeRecords?login=%u&passwd=%p&input_format=json&output_format=json&input_data={"fqdn":"%h","records":{"A":[{"priority":10,"value":"%i"}],"MX":[{"priority":10,"value":"mx1.beget.com"},{"priority":20,"value":"mx2.beget.com"}],"TXT":[{"priority":10,"value":"v=spf1 redirect=beget.com"}]}}'
	hostname = host.name
}

Does anyone had something related or similar?

PS: for other hosts i can use short one ddns-path, but one host require to provide MX and TXT records. Beget.API is overwriting old MX and TXT with blanks if they are not provided.
PPS: API requests were firstly tested in browser - both works, but not in inadyn :(

[vdubikov]

/include/ddns.h

#define SERVER_URL_LEN                    256     /* chars */

changed to 512 and it works now.

[troglobit]

Pull request to address this, either by extending the static default, or changing to dynamic allocation, are most welcome.

[vdubikov]

Done.

[troglobit]

Awesome, thanks!