need to convert from json string to dict when extracting message from event

[alexgottscha]

The value of event['Records'][0]['Sns']['Message'] is a json string, not a dictionary. I was getting this error prior to making the change:

string indices must be integers: TypeError
Traceback (most recent call last):
File "/var/task/notify_slack.py", line 79, in lambda_handler
notify_slack(subject, message, region)
File "/var/task/notify_slack.py", line 64, in notify_slack
payload['text'] = "AWS CloudWatch notification - " + message["AlarmName"]
TypeError: string indices must be integers

[antonbabenko]

@invertigo We went back and forth with a similar change I believe. See #24, #23, #25 and maybe few more.

Can somebody also take a look at this/these?

[alexgottscha]

@invertigo We went back and forth with a similar change I believe. See #24, #23, #25 and maybe few more.

Can somebody also take a look at this/these?

I updated the patch here based on the changes you linked - should take care of most cases and provide a meaningful error if not.

[antonbabenko]

@invertigo 🤞 Ok, let's merge and see.

@ryron01 Do you have an opinion on this?

[antonbabenko]

v1.13.0 has been released.

[github-actions]

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.