Skip to content
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

TopicARN has to be an array of strings in AWS::AutoScaling::AutoScalingGroup NotificationConfiguration #2929

Open
ivorobioff opened this issue Oct 25, 2023 · 1 comment
Labels
v1 v1.X

Comments

@ivorobioff
Copy link

CloudFormation Lint Version

0.83.0

What operating system are you using?

ubuntu

Describe the bug

Even though the official documentation tells that TopicARM has to be an array of string the us-east-1.json specifies that it has to be string.

Official Documentation https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-autoscaling-autoscalinggroup-notificationconfiguration.html#cfn-autoscaling-autoscalinggroup-notificationconfiguration-topicarn

Screenshot_25

Expected behavior

after running cfn-lint template.yaml no error has to be with the following configuration

Untitled

Reproduction template

unfortunately cannot provide one

@kddejong
Copy link
Contributor

kddejong commented Oct 30, 2023

Yea this is a weird scenario. It accepts both as long as the list is just one. #2914

This will be fixed in v1 as we be able to support multiple types. At this time the CloudFormation specs we rely on only support one type. This type was originally a string so that is why we are favoring that type over the list.

This also affects the docs as the docs will pick one type.

@kddejong kddejong added the v1 v1.X label Jan 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v1 v1.X
Projects
None yet
Development

No branches or pull requests

2 participants