contrib/aws: fix sns nil pointer dereference

[satorunooshie]

What does this PR do?

Fixes bugs since v1.51.0

https://github.com/aws/aws-sdk-go-v2/blob/fa15a743cc4b93dfc290d2622c92b3969ca004cf/service/sns/api_op_Publish.go#L114-L133

	// The phone number to which you want to deliver an SMS message. Use E.164 format.
	// If you don't specify a value for the PhoneNumber parameter, you must specify a
	// value for the TargetArn or TopicArn parameters.
	PhoneNumber *string

	// Optional parameter to be used as the "Subject" line when the message is
	// delivered to email endpoints. This field will also be included, if present, in
	// the standard JSON messages delivered to other endpoints. Constraints: Subjects
	// must be ASCII text that begins with a letter, number, or punctuation mark; must
	// not include line breaks or control characters; and must be less than 100
	// characters long.
	Subject *string

	// If you don't specify a value for the TargetArn parameter, you must specify a
	// value for the PhoneNumber or TopicArn parameters.
	TargetArn *string

	// The topic you want to publish to. If you don't specify a value for the TopicArn
	// parameter, you must specify a value for the PhoneNumber or TargetArn parameters.
	TopicArn *string

Motivation

Fixes #2001

Describe how to test/QA your changes

Reviewer's Checklist

• Changed code has unit tests for its functionality.
• If this interacts with the agent in a new way, a system test has been added.

[zARODz11z]

Hi @satorunooshie, thank you for the contribution! Adding the phone number seems out of scope for fixing #2001 due to privacy concerns that we'd need to reevaluate further. Lets remove the addition of the phone number tag for now.

[satorunooshie]

@zARODz11z Hello, thanks for your review. Actually I was wondering how I should handle the phone number.
fixed at 5aa235b

[zARODz11z]

@satorunooshie Thank you for the changes! That is what I was looking for.

[zarirhamza]

LGTM as well. The go tracer team should review shortly