Description

This commit fixes the previous behavior of dynamicbeat crashing when an invalid check is parsed. This is called from dynamicbeat setup checks and would happen when adding checks to elastic, by accessing a null pointer. As the error is passed to Errorf() this still by zap's defaults results in a large stack trace, but dynamicbeat will try to upload the next check as expected. Disabling stack traces on logs like this is possible using the documentation here.

Fixes #305

Merge Checklist

• I have tested this change locally to make sure it works
• I have updated the documentation as necessary
• I have added a release note under the Unreleased section of the Changelog
• Any relevant labels have been added
• This PR is being merged into dev, unless it's a PR for a release