pulumi / pulumi-eks

A Pulumi component for easily creating and managing an Amazon EKS Cluster
https://www.pulumi.com/registry/packages/eks/
Apache License 2.0
171 stars 80 forks source link

Fix undefined panic when using `.map` #1214

Closed rquitales closed 3 months ago

rquitales commented 3 months ago

Proposed changes

Adds an undefined guard before attempting to use .map(). This is due to MLC programs being able to send undefined instead of an empty array.

This PR also updates the error handling to throw pulumi.ResourceError instead of a generic nodejs error.

Related issues (optional)

Fixes: #1202

github-actions[bot] commented 3 months ago

Does the PR have any schema changes?

Looking good! No breaking changes found. No new resources/functions.

github-actions[bot] commented 3 months ago

Does the PR have any schema changes?

Looking good! No breaking changes found. No new resources/functions.

github-actions[bot] commented 3 months ago

Does the PR have any schema changes?

Looking good! No breaking changes found. No new resources/functions.

github-actions[bot] commented 3 months ago

Does the PR have any schema changes?

Looking good! No breaking changes found. No new resources/functions.