inwx: Be case insensitive while searching for the cookie.

[sebastianas]

At least since 2023-08-25 the cookie is set via set-cookie' instead the expecting Set-Cookie' string. A month earlier it was working.

Ignore the case while matching the cookie.

[github-actions]

Welcome
Please make sure you're read our DNS API Dev Guide and DNS-API-Test.
Then reply on this message, otherwise, your code will not be reviewed or merged.
We look forward to reviewing your Pull request shortly ✨

[sebastianas]

Welcome Please make sure you're read our DNS API Dev Guide and DNS-API-Test. Then reply on this message, otherwise, your code will not be reviewed or merged. We look forward to reviewing your Pull request shortly ✨

This should work out.

[spaetz]

Not tested yet, but thanks already. Not sure if it would help as I don't seem to be getting any Header back, just a HTtP/2. But I will try your patch in any case.

[inwx-developer]

We have tested the patch and it's working.

[haupas]

👍 Strongly Support This Pull Request 👏

I'd like to express our strong support for this pull request. It addresses a critical bug that has been affecting our company and numerous customers, causing significant disruptions.

We've been awaiting a solution to this issue and have conducted thorough testing on this pull request. I'm pleased to confirm that it effectively resolves the problem. Given the substantial impact this bug has had, I believe that merging this pull request is of the utmost importance.

Additionally, I'd like to highlight that we have a dependency on the acme.sh software in several systems throughout our infrastructure. The process of updating these packages will take some additional time, and the sooner we can get this pull request merged, the sooner we can begin that process.

Thank you for considering our input, and we sincerely hope to see this pull request merged soon.

Best regards

[zeroservices]

+1

Would really appreciate this pull request being merged soon.

[bekanne]

Will fix this issue: #4763

[haupas]

Thank's for merging the pull request into the dev branch.
When can we except this fix to appear in the main branch / inside a new release?
Currently we fixed the issue manually in some of our systems, where certitifcates would expire soon. But we would rather not like this to be necessary on all our systems, which use the acme client.

[Neilpang]

It will be one week or 2.

[TheHellSite]

Hello @Neilpang ,

any news on the ETA for the release including the pull request?

[Neilpang]

here you go:
https://github.com/acmesh-official/acme.sh/releases/tag/3.0.7