aws-cloudformation / aws-cloudformation-resource-providers-logs

The CloudFormation Resource Provider Package For Amazon CloudWatch Logs
https://docs.aws.amazon.com/AmazonCloudWatch/latest/logs/WhatIsCloudWatchLogs.html
Apache License 2.0
33 stars 35 forks source link

Fix Log group update check NPE issue #49

Closed wbingli closed 4 years ago

wbingli commented 4 years ago

Issue #, if available:

Description of changes:. We have NPE issue when update but previous source without retention or Kms configured.

Also include unit test to check the API call request to ensure for such case.

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