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

Incorrect check for config items when sending reload failures to New Relic #596

Closed
mountaindude opened this issue Nov 28, 2022 · 0 comments · Fixed by #601 or #981
Closed

Incorrect check for config items when sending reload failures to New Relic #596

mountaindude opened this issue Nov 28, 2022 · 0 comments · Fixed by #601 or #981
Assignees
Labels

Comments

@mountaindude
Copy link
Collaborator

What version of Butler are you using?

8.6.0

What version of Node.js are you using? Not applicable if you use the standalone version of Butler.

No response

What command did you use to start Butler?

N/A

What operating system are you using?

Windows, macOS

What CPU architecture are you using?

x86_64

What Qlik Sense versions are you using?

2021-Aug

Describe the Bug

When Butler sends reload failures and events and log entries to New Relic, it does this check:

image

I.e. checks if either of those two config entries exist.

That is incorrect.
It is only if both those config entries exist that the custom property's value should be used as destination for events / log entries.

Note that this applies to both New Relic events and log entries.

Expected Behavior

No response

To Reproduce

No response

@mountaindude mountaindude self-assigned this Nov 28, 2022
@mountaindude mountaindude changed the title Incorrect check for config items when sending reload failures to New Relic§ Incorrect check for config items when sending reload failures to New Relic Dec 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment