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

Invoke-icingacheckhttpstatus - unknown #286

Closed
haytxy opened this issue Feb 21, 2022 · 4 comments · Fixed by #287
Closed

Invoke-icingacheckhttpstatus - unknown #286

haytxy opened this issue Feb 21, 2022 · 4 comments · Fixed by #287
Assignees
Labels
bug Something isn't working
Milestone

Comments

@haytxy
Copy link

haytxy commented Feb 21, 2022

hello,

testing with curl gives me http 200.
image

using Invoke-icingacheckhttpstatus gives me an unknown for response time.
image

any idea how to approach this issue.

mj

@haytxy
Copy link
Author

haytxy commented Feb 21, 2022

tested locally
image

image

image

@LordHepipud LordHepipud self-assigned this Feb 22, 2022
@LordHepipud LordHepipud added the bug Something isn't working label Feb 22, 2022
@LordHepipud LordHepipud added this to the v1.9.0 milestone Feb 22, 2022
@LordHepipud
Copy link
Collaborator

Thank you for the issue. I will have a look on this.
Does this happen only to this address or is it a general problem with the plugin and all websites?

@stevie-sy
Copy link

@LordHepipud the author also created a thread in the community https://community.icinga.com/t/invoke-icingacheckhttpstatus-unknown/9281/2

For me it looks like the "problem" is the subroutine Get-IcingaCheckHTTPQuery. You alway run Start-IcingaTimer, but you call Get-IcingaTimer only if there is a content.

In my eyes you should run
$HTTPData.Add('RequestTime', (Get-IcingaTimer -Name 'HTTPRequest').Elapsed.TotalSeconds);
always.

Why there is no return content, only the author can answer. ... just my 5 cents :-)

@LordHepipud
Copy link
Collaborator

Thanks, yes - that was the problem

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants