Uses affected accounts

[O5m4r]

Issue #, if available:

Some parameters that were required by some functions were not being passed

Description of changes:

Passed parameters to functions to successfully send notifications

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[rm-fr]

["errorMessage": "name 'affected_accounts' is not defined"]

I got the above error, but
I was able to fix that error with the source you fixed.
thank you.

However, the following error is additionally occurring.

{
"errorMessage": "get_message_for_email() missing 2 required positional arguments: 'affected_accounts' and 'affected_entities'",
"errorType": "TypeError",
"stackTrace": [
" File \"/var/task/handler.py\", line 837, in main\n describe_events(health_client)\n",
" File \"/var/task/handler.py\", line 687, in describe_events\n update_ddb(event_arn, str_update, status_code, event_details, affected_accounts, affected_entities)\n",
" File \"/var/task/handler.py\", line 491, in update_ddb\n send_alert(event_details, affected_accounts, affected_entities, event_type=\"resolve\")\n",
" File \"/var/task/handler.py\", line 97, in send_alert\n send_email(event_details, event_type)\n",
" File \"/var/task/handler.py\", line 234, in send_email\n BODY_HTML = get_message_for_email(event_details, eventType)\n"
]
}

[jonjohnston]

I think they implemented your fix
@rm-fr I opened issue for the email error but you can fix it yourself by changing in the handler.py def send_email function to include affected_accounts, affected_entities

def send_email(event_details, event_type, affected_accounts, affected_entities):

[jordanaroth]

This should be fixed now