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

Empty http header array for New Relic uptime storage results in run-time error #562

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

Empty http header array for New Relic uptime storage results in run-time error #562

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

Comments

@mountaindude
Copy link
Collaborator

What version of Butler are you using?

8.5.1

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?

butler.exe -c D:\butler\config\butler-config.yaml --new-relic-account-name ACCOUNTNAME --new-relic-account-id ACCOUNTID --new-relic-api-key APIKEY

What operating system are you using?

Windows, macOS

What CPU architecture are you using?

x86_64

What Qlik Sense versions are you using?

2022 May

Describe the Bug

If the http header section in the config file is empty, a run-time error will occur when Butler tries to store uptime info to New Relic.

    storeNewRelic:
      enable: true
      destinationAccount:
        - SOME_ACCOUNT
      url: https://insights-collector.eu01.nr-data.net/metric/v1   # Where should uptime data be sent?
      header:                       # Custom http headers
        #- name: X-My-Header
        #  value: Header value

The error is:

error: UPTIME NEW RELIC: Error sending uptime: TypeError: globals.config.get is not a function or its return value is not iterable

Expected Behavior

Should be possible to have empty arrays for http headers in the config file.

To Reproduce

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment