-
Notifications
You must be signed in to change notification settings - Fork 0
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
Slack webhook used in workflows no longer working #1737
Labels
Comments
Admin has been fixed in cds-snc/notification-admin#2041 |
This was referenced Jan 10, 2025
API fix: cds-snc/notification-api#2413 |
5 tasks
document-download-api: cds-snc/notification-document-download-api#209 |
5 tasks
notification-documentation: cds-snc/notification-documentation#171 |
5 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug
The slack webhook used in a number of workflows is no longer working due to invalid JSON. Nothing has changed here, but it looks as though we are using single quotes in our JSON instead of double quotes - which is not valid. It is possible that the slack code has been updated recently to be more strict.
Potentially affected workflows:
notification-admin
s3-backup.ymltest_prod_config.yamltest.yamlnotification-adr
s3-backup.ymlnotification-api
s3-backup.ymlnotification-document-download-api
s3-backup.ymlnotification-documentation
s3-backup.ymlnotification-lambdas
s3-backup.ymlnotification-manifests
s3-backup.ymlnotification-system-status-frontend
prod_upload_to_s3.ymlupload_to_s3.ymlnotification-terraform
check_eks_ami_update.ymlcheck_eks_cluster_update.ymlcheck_rds_cluster_update.ymls3-backup.ymlworkflow_failure.ymlnotification-utils
s3-backup.ymlSEV-2 Major
Expected behavior
When a workflow fails, a notification should be sent to slack to alert us.
Impact
Describe overall user/system impact to consider when prioritizing this issue.
If applicable
Impact on Notify users:
Impact on Recipients:
Impact on Notify team:
The text was updated successfully, but these errors were encountered: