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
Adds HTTP integration URIs #8569
Adds HTTP integration URIs #8569
Conversation
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! This is awesome, thanks for adding this feature so quickly, sorry for the delay in reviewing.
It would be nice if we could validate it against AWS, even if it seems obvious this will work against it, just so that we have the marker and we have confidence it runs (also for maybe future automated tests against AWS). I've left a comment regarding a fixture that can be used to run against AWS as well.
Thanks again! Also, awesome use of the import feature that makes it so much simpler to set up simple resources instead of so many direct SDK calls. 🚀
(cherry picked from commit c239fc0)
97ee786
to
70a76cd
Compare
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 🚀 - kudos @calvernaz for adding the AWS validated test.
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.
Awesome, thanks for making it AWS validated, sorry for scheme issue. 🚀
Thank you all! |
Changes
Adds support for HTTP integration URIs https://docs.aws.amazon.com/apigateway/latest/developerguide/aws-api-gateway-stage-variables-reference.html