-
Notifications
You must be signed in to change notification settings - Fork 145
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
publish AHA v2.3 #105
Merged
publish AHA v2.3 #105
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Fix for template file / env vars / eventbridge schedule Merging. Will notify requestors of fixes and request feedback.
Issue 76 Fixed Terraform script to include an Environment Variable for the configured endpoint
…ng logic (#90) * Add support for Slack 'triggers' webhooks - streamline webhook handling logic --------- Co-authored-by: Andrew Riley <rileyac@amazon.com>
Co-authored-by: Andrew Riley <rileyac@amazon.com>
Chime has a 4096 byte limit for sent messages. Truncate any longer messages and indicate that the message is truncated. Fixes #70
To reduce the number of boto3 clients that are created we introduce AWSApi which uses functools.lru_cache to cache boto3 clients. This can significantly reduce the lambda runtime.
Reduce the number of times we need to call the AWS api to fetch secrets by caching the response.
Fixes/general
update to Slack webhook/trigger handling
Fixes/general
Updates/webhooks
Updates/terraform
Updates/docs
Add release 2.3 notes
Merge release 2.3 notes, doc updates for delegated admin, exclude account ids +
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of changes:
Performance Improvements
from #93
Cache boto3 client calls @joyofhex in 6e8e841
Cache Secrets Manager calls by @joyofhex& c2ac41f
Terraform updates
Fix for template file / env vars / eventbridge schedule by @davedimm in #88
Issue 76 Fixed Terraform script to include an Environment Variable for the configured endpoint by @calorie in #84
Slack Workflow 2.0 webhook support
Add support for Slack 'triggers' webhooks - streamline webhook handling logic by @andrewcr7 in #90
update to Slack webhook/trigger handling by @andrewcr7 in #94
General Fixes
from #93
Truncate long chime messages by @joyofhex in aae649d
Handle empty endTime in certain events by @joyofhex in 5cca8fb
Documentation updates
from #101, update docs for
Sample ExcludeAccountIDs file and troubleshooting notes by @andrewcr7 in c3da096 & ccd1396
Using AHA with AWS Health Delegated Administrator @andrewcr7 in update delegated admin notes and TOC
Organizational View notes and link to documentation by @andrewcr7 in 0378ff6
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.