(cfn-include): AWS::NoValue fails for a boolean-typed property

[skinny85]

What is the problem?

When trying to include a template with the cloudformation-include module that uses { Ref: 'AWS::NoValue' } for a boolean-typed property, you get an error like:

Error: Expected 'true' or 'false' for boolean value, got: '${Token[AWS.NoValue.7]}'

Reproduction Steps

Here's a minimal template showing the issue:

AWSTemplateFormatVersion: "2010-09-09"
Resources:
  CloudFrontDistribution:
    Type: "AWS::CloudFront::Distribution"
    Properties:
      DistributionConfig:
        Enabled: "true"
        ViewerCertificate:
          CloudFrontDefaultCertificate:
            Ref: "AWS::NoValue"

[skinny85]

The problem is here. Our CFN parser uses the built-in Aws.NO_VALUE constant, which erroneously assumes that AWS::NoValue can only be used for a string, but of course NoValue can be used for any type, including complex types.

So, the solution is to change this line to use an IResolvable instead of Aws.NO_VALUE, and this way we'll never get into this branch of the getBoolean() function, and everything will work great.

[github-actions]

This issue has not received any attention in 1 year. If you want to keep this issue open, please leave a comment below and auto-close will be canceled.

[skinny85]

@comcalvi should I post a fix for this? It should be easy, as I explained in this comment.