[BUG] activation module with redirect set to true fails if API returns a 302 (possibly only HTTPS) about ansible-collection-checkmk.general HOT 6 CLOSED

Hi Beni,

thank you very much for this very well described problem.
I will look into this. As you already wrote the steps to reproduce this need some work, so please don't expect a solution within a few days.

I would suggest not to remove this option, but to handle the redirect properly.
If it's a bug in the API, we have to fix it there anyway.

KR,
Max

from ansible-collection-checkmk.general.

Hi @hb9hnt,

i know that your example is about the activation endpoint, but it looks like the issue we already have with #414 is the same. Can you confirm this?

KR,
Max

from ansible-collection-checkmk.general.

Hey @Max-checkmk

I do agree that the cause of the issue is the same: calling an API endpoint which possibly returns a 302 to a background job. However I don't agree that the two issues are the same, because the discovery endpoint domain-types/service_discovery_run/actions/start/invoke doesn't give you the option to set redirect: false and make the request synchronous.

IMHO solving this issue is a lot easier than fixing #441 and the resolution that makes sense for #441 might not applicable for the activation endpoint described here.

However, you know the inner workings of Checkmk way better than I do. If you already see a solution that's applicable to both endpoints I'm interested in how you'd solve it.

--Beni

from ansible-collection-checkmk.general.

Hi Beni,

i just had a customer ticket where we had the exact same problem.
The API answers with the same protocol in which the request came in.
In this case the system apache wasn't configured to forward the https to the site apache.

Could you please check if your system apache is configured like described here?
https://docs.checkmk.com/latest/en/omd_https.html?lquery=apache#httpsredirect

It's about those lines:
<IfModule headers_module>
RequestHeader set X-Forwarded-Proto expr=%{REQUEST_SCHEME}
RequestHeader set X-Forwarded-SSL expr=%{HTTPS}
</IfModule>

KR,
Max

from ansible-collection-checkmk.general.

Hey Max

I can confirm that this header was missing from my config. I added the header. At the moment I don't have the opportunity to test this, but I think we can assume that this issue doesn't need fixing.

Thank you and regards

Beni

from ansible-collection-checkmk.general.

Thanks for the update guys! 🙏

I will close this issue then for now, if the issue remains, feel free to re-open it @hb9hnt. ✌️

from ansible-collection-checkmk.general.