Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: spaces in FnSub literals are causing value to not be a literal #358

Merged
merged 2 commits into from
Oct 13, 2022

Conversation

mrgrain
Copy link
Contributor

@mrgrain mrgrain commented Oct 13, 2022


Template simplified for brevity.

{
  "Resources": {
    "Bucket": {
      "Type": "AWS::IAM::Role",
      "Properties": {
        "Description": { "Fn::Sub": "${!NoSpace}${ ! BothSpace}${! AfterSpace}${ !BeforeSpace}${!EndSpace }" },
      }
    }
  }
}

image

@mrgrain mrgrain marked this pull request as draft October 13, 2022 16:17
@mrgrain mrgrain force-pushed the mrgrain/fix/fnsub-literal-spaces branch from 1d1f690 to 34ab60f Compare October 13, 2022 17:02
@mrgrain mrgrain force-pushed the mrgrain/fix/fnsub-literal-spaces branch from 34ab60f to 0f736b5 Compare October 13, 2022 17:06
@mrgrain mrgrain marked this pull request as ready for review October 13, 2022 17:07
@mergify mergify bot merged commit 1bb1ad2 into main Oct 13, 2022
@mergify mergify bot deleted the mrgrain/fix/fnsub-literal-spaces branch October 13, 2022 17:17
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants