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

AwsCustomResource doesn't regard lists for the outputPath parameter #9596

Closed
mmuller88 opened this issue Aug 11, 2020 · 0 comments
Closed

AwsCustomResource doesn't regard lists for the outputPath parameter #9596

mmuller88 opened this issue Aug 11, 2020 · 0 comments
Labels
bug This issue is a bug. needs-triage This issue or PR still needs to be triaged.

Comments

@mmuller88
Copy link
Contributor

Relates to #2825 where you could reduce the response using outputPath. That filter only applies for objects but responses can have lists as well. I am runnint into the 4k limit issue with accessing https://docs.aws.amazon.com/cli/latest/reference/ec2/describe-vpn-connections.html where that are lists.

Reproduction Steps

  1. Create a Custom Resource with accessing https://docs.aws.amazon.com/cli/latest/reference/ec2/describe-vpn-connections.html
new AwsCustomResource(this, 'describeVpnConnections', {
            onCreate: {
                service: 'EC2',
                action: 'describeVpnConnections',
                parameters: {
                    VpnConnectionIds: [vpnConnection.vpnId]
                },
                outputPath: 'VpnConnection.Options',
                physicalResourceId: PhysicalResourceId.of(`${vpnConnection.vpnId}-describe`),
            },
            policy: AwsCustomResourcePolicy.fromSdkCalls({resources: AwsCustomResourcePolicy.ANY_RESOURCE})
        });

What did you expect to happen?

Response successfull and only Options are returned.

What actually happened?

Fails as response ist too long

Environment

  • CLI Version : 1.57
  • Framework Version:
  • Node.js Version:
  • OS :
  • Language (Version):

Other


This is 🐛 Bug Report

@mmuller88 mmuller88 added bug This issue is a bug. needs-triage This issue or PR still needs to be triaged. labels Aug 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug. needs-triage This issue or PR still needs to be triaged.
Projects
None yet
Development

No branches or pull requests

1 participant