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.

Sign up for GitHub

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

Jump to bottom

Add API Gateway #10

Merged
merged 2 commits into from
Oct 23, 2023
Merged

Add API Gateway #10

merged 2 commits into from
Oct 23, 2023

Conversation

gregtyler
Copy link
Contributor

Use OpenAPI schema to define REST API, then add logging and IAM execution permissions.

Remove lambda function URLs. Test against REST API base URL.

Fixes VEGA-2014 #minor

@gregtyler gregtyler temporarily deployed to 10vega2104a popup environment October 20, 2023 15:09 — with GitHub Actions Inactive
@gregtyler gregtyler force-pushed the VEGA-2104-add-apigw branch from 5e20b35 to 61a0c61 Compare October 20, 2023 15:23
@gregtyler gregtyler temporarily deployed to 10vega2104a popup environment October 20, 2023 15:26 — with GitHub Actions Inactive
@gregtyler gregtyler force-pushed the VEGA-2104-add-apigw branch from 61a0c61 to 646e160 Compare October 20, 2023 15:52
@gregtyler gregtyler temporarily deployed to development account October 20, 2023 15:52 — with GitHub Actions Inactive
@gregtyler gregtyler temporarily deployed to 10vega2104a popup environment October 20, 2023 15:55 — with GitHub Actions Inactive
@gregtyler gregtyler force-pushed the VEGA-2104-add-apigw branch from 646e160 to a3b5a5c Compare October 20, 2023 15:56
@gregtyler gregtyler temporarily deployed to development account October 20, 2023 15:56 — with GitHub Actions Inactive
@gregtyler gregtyler temporarily deployed to 10vega2104a popup environment October 20, 2023 15:58 — with GitHub Actions Inactive
@gregtyler gregtyler temporarily deployed to development account October 20, 2023 16:00 — with GitHub Actions Inactive
@gregtyler gregtyler temporarily deployed to 10vega2104a popup environment October 20, 2023 16:00 — with GitHub Actions Inactive
@gregtyler gregtyler force-pushed the VEGA-2104-add-apigw branch from a3b5a5c to 87bc026 Compare October 20, 2023 16:08
@gregtyler gregtyler temporarily deployed to development account October 20, 2023 16:09 — with GitHub Actions Inactive
@gregtyler gregtyler temporarily deployed to 10vega2104a popup environment October 20, 2023 16:11 — with GitHub Actions Inactive
@gregtyler gregtyler force-pushed the VEGA-2104-add-apigw branch from 87bc026 to ce84634 Compare October 20, 2023 16:17
@gregtyler gregtyler temporarily deployed to development account October 20, 2023 16:17 — with GitHub Actions Inactive
@gregtyler gregtyler temporarily deployed to 10vega2104a popup environment October 20, 2023 16:19 — with GitHub Actions Inactive
@gregtyler gregtyler force-pushed the VEGA-2104-add-apigw branch from ce84634 to 1711202 Compare October 20, 2023 16:33
@gregtyler gregtyler temporarily deployed to development account October 20, 2023 16:33 — with GitHub Actions Inactive
@gregtyler gregtyler temporarily deployed to 10vega2104a popup environment October 20, 2023 16:36 — with GitHub Actions Inactive
@gregtyler gregtyler force-pushed the VEGA-2104-add-apigw branch from 1711202 to ee5ea43 Compare October 20, 2023 16:39
@gregtyler gregtyler temporarily deployed to development account October 20, 2023 16:39 — with GitHub Actions Inactive
@gregtyler gregtyler temporarily deployed to 10vega2104a popup environment October 20, 2023 16:42 — with GitHub Actions Inactive
@gregtyler gregtyler force-pushed the VEGA-2104-add-apigw branch from ee5ea43 to 52371f6 Compare October 20, 2023 16:45
@gregtyler gregtyler temporarily deployed to development account October 20, 2023 16:45 — with GitHub Actions Inactive
@gregtyler gregtyler temporarily deployed to 10vega2104a popup environment October 20, 2023 16:47 — with GitHub Actions Inactive
@gregtyler gregtyler force-pushed the VEGA-2104-add-apigw branch from 52371f6 to a5aac23 Compare October 20, 2023 17:01
@gregtyler gregtyler temporarily deployed to development account October 20, 2023 17:01 — with GitHub Actions Inactive
@gregtyler gregtyler temporarily deployed to 10vega2104a popup environment October 20, 2023 17:03 — with GitHub Actions Inactive
@gregtyler
Add API Gateway
50050f3 
Use OpenAPI schema to define REST API, then add logging and IAM execution permissions.

Add an account-level IAM role for logging to CloudWatch.

Remove lambda function URLs. Test against REST API base URL.

Fixes VEGA-2014 #minor
@gregtyler gregtyler force-pushed the VEGA-2104-add-apigw branch from a5aac23 to 50050f3 Compare October 20, 2023 17:13
@gregtyler gregtyler temporarily deployed to 10vega2104a October 20, 2023 17:16 — with GitHub Actions Inactive
@gregtyler
Remove function URLs
4b93954 
Not needed now we use API Gateway

#patch
@gregtyler gregtyler marked this pull request as ready for review October 23, 2023 09:36
@gregtyler gregtyler requested a review from a team as a code owner October 23, 2023 09:36
@gregtyler gregtyler merged commit 90e5d8c into main Oct 23, 2023
8 checks passed
@gregtyler gregtyler deleted the VEGA-2104-add-apigw branch October 23, 2023 09:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sixdaysandy sixdaysandy sixdaysandy approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gregtyler @sixdaysandy