-
Notifications
You must be signed in to change notification settings - Fork 249
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
aws-apigateway-kinesisstreams based on an OpenAPI specification #913
Comments
Hello @georgebearden, I see that you are working on #912. Would it be possible to have the same also for API GW + Kinesis? |
We'll take a look. Fulfilling #912 in a way that works best for client apps is going to be one of our most complex implementations to date, but we are hopeful we've found a way forward. |
Do you accept PRs? Can you share more details about your implementation? |
Our goal is to be able to store your API definition file separate from your CDK app code - the definition will be an asset so can reside locally or in S3. The generated CloudFormation will reference the asset in the CDK staging buckets - this is important, as including the API spec in the generated CloudFormation template can grow your template past the maximum size. The problem here is referencing the newly launched Lambda functions in the asset. Regular CDK tokens will not be substituted if the file is referenced rather than imported. Our current thought is to write a custom resource that loads the API Spec after the Lambda function is generated and substituting the arn of the new functions in the uri field as the last step. We're running some test code to see if we can make that work. |
After reading this issue, we're also looking at how the substitution could be general enough that it could be applicable for resources other than Lambda functions - e.g. new Kinesis Streams. |
On the line of #840 and #912 it would be great to get the features also for
aws-apigateway-kinesisstreams
Use Case
Be able to create an API Gateway using OpenAPI specifications
Proposed Solution
Create construct
aws-openapigateway-kinesisstreams
Other
This is a 🚀 Feature Request
The text was updated successfully, but these errors were encountered: