-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
Fix issue with parsing out endpoint package policy. #123287
Conversation
Pinging @elastic/security-solution (Team: SecuritySolution) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@@ -145,11 +146,11 @@ export function createTelemetryEndpointTaskConfig(maxTelemetryBatch: number) { | |||
packagePolicies | |||
.map((pPolicy) => pPolicy as PolicyData) | |||
.forEach((pPolicy) => { | |||
if (pPolicy.inputs[0].config !== undefined) { | |||
if (pPolicy.inputs[0]?.config !== undefined && pPolicy.inputs[0]?.config !== null) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Gotcha, can see how this would error.
💚 Build Succeeded
Metrics [docs]
To update your PR or re-run it, just comment with: cc @pjhampton |
* Fix issue with parsing out endpoint package policy. * revert schedule. (cherry picked from commit 4d98af8)
💔 Some backports could not be created
How to fixRe-run the backport manually:
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
* Fix issue with parsing out endpoint package policy. * revert schedule. (cherry picked from commit 4d98af8) Co-authored-by: Pete Hampton <[email protected]>
Summary
We found an edge case on customer deployments where the telemetry was going dark due to a type error. While the root cause of why this is happening hasn't been established, the optional chaining and defaulting circumvents the reproducible type error.
Checklist
Delete any items that are not applicable to this PR.
Risk Matrix
Delete this section if it is not applicable to this PR.
Before closing this PR, invite QA, stakeholders, and other developers to identify risks that should be tested prior to the change/feature release.
When forming the risk matrix, consider some of the following examples and how they may potentially impact the change:
For maintainers