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

No validation on AWS::Events::Rule Name length #3934

Closed
AriYacobson opened this issue Jan 29, 2025 · 1 comment · Fixed by #3939
Closed

No validation on AWS::Events::Rule Name length #3934

AriYacobson opened this issue Jan 29, 2025 · 1 comment · Fixed by #3939

Comments

@AriYacobson
Copy link

CloudFormation Lint Version

1.22.7

What operating system are you using?

Windows

Describe the bug

When running cfn-lint on a CF yaml template
The cfn-lint doesn't validate that the length of the AWS::Events::Rule Name property is less than 64 chars
as mentioned here

Expected behavior

cfn-lint to return an E3033 error saying '' is longer than 64

Reproduction template

AWSTemplateFormatVersion: 2010-09-09
Description: >-
  Rule that listens for state changes of CloudWatch Alarms rule name too long

Resources:
  EventRule:
    Type: AWS::Events::Rule
    Properties:
      Name: 'RuleNameTooLong0000000000000000000000000000000000000000000000000000'
      Description: 'Event rule that listens for state changes of CloudWatch Alarms'
      EventPattern:
        source:
          - "aws.cloudwatch"
        detail-type:
          - "CloudWatch Alarm State Change"
        detail:
          alarmName:
            - "Alarm1"
      State: ENABLED
      Targets:
        - Arn: arn:aws:cloudwatch:us-west-2:123456789012:alarm:Alarm1
          Id: "LogGroupTarget"```
@kddejong
Copy link
Contributor

Documented in the API

Going to look into our boto automation as its defined in botocore definitions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants