-
-
Notifications
You must be signed in to change notification settings - Fork 5k
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
Setting of txt-records doesn't work on INWX Api #4763
Comments
Please upgrade to the latest code and try again first. Maybe it's already fixed. |
Here's the log with --debug 2: [Mon 28 Aug 10:16:35 CEST 2023] Running cmd: renew |
I found the problem in the dns_inwx.sh script. The cookie string cannot be saved because INWX changed a header key to lower case. This string is needed to stay authenticated for all further requests to the INWX API. Solution: Line 197 in 0da839c
So the line should be look like this: @Kulturserver you can also use this as workaround by editing the .acme.sh/dnsapi/dns_inwx.sh file. |
Thank you very much! With this change we were able to trigger the update. Maybe this should be patched in the script directly? |
Issue should remain open until it is patched here. As I understand it right, the header filed name is case-insensitive. |
ok, sorry, it's reopened now! |
Hey there, we just wanted to report this issue. While header fields always were case-insensitive our infrastructure used "Set-Cookie" in its responses. https://datatracker.ietf.org/doc/html/rfc9113#section-8.2.1 Best regards |
Thanks to @Kulturserver for reporting this issue. Also huge thanks to @bekanne for checking the code and finding the problem and suggest a workaround! |
Hi, we've updated to the newest acme.sh Version 3.0.7 and still encounter a problem with setting the txt record on the INWX Api - it isn't possible and so the certificates cannot be extended. A week ago everything worked. Can anybody help? The log file is below.
The text was updated successfully, but these errors were encountered: