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

Api rewrite #10

Merged
merged 15 commits into from
Sep 18, 2021
Merged

Api rewrite #10

merged 15 commits into from
Sep 18, 2021

Conversation

c6fc
Copy link
Owner

@c6fc c6fc commented Sep 18, 2021

Merge api_rewrite into dev. This includes multiple changes:

  • Terraform version bump to 0.15
  • Improvements in lambda function readability and error handling; async/await with try-catch blocks on each segment:
    • rewrite of API gateway generator and break-out of proxy_api_handler
    • rewrite of spot_monitor
  • Dockerfile bumped to AWSCLIv2, with quality-of-life changes to make existing deployments easier to manage
    • Docker now contains all the prereqs for self-hosted deployments, so advanced deployments can be entirely managed via Docker.
  • Multiple redundant settings were removed, such as 'useSAML', 'useCustomDns', and 'dnsNames'. These are now assumed based on whether a SAML Metadata file or Route53 zone ID is provided, and the deployment URL is now derived from the Route53 Zone.
  • SAML login is now supported without custom DNS, and works correctly with events and settings.

@c6fc c6fc merged commit 61c72d5 into dev Sep 18, 2021
@c6fc c6fc deleted the api_rewrite branch November 16, 2021 19:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants