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

Issue parsing colons on Linux #1984

Closed
farski opened this issue Apr 15, 2021 · 5 comments
Closed

Issue parsing colons on Linux #1984

farski opened this issue Apr 15, 2021 · 5 comments

Comments

@farski
Copy link

farski commented Apr 15, 2021

cfn-lint version: 0.48.2 (cfn-lint --version)

Description of issue.

When I lint this file on my Mac using 0.48.2, I get no errors. When the same file is linted using 0.48.2 in CodeBuild (aws/codebuild/amazonlinux2-x86_64-standard:3.0) it returns:

E0000 found unexpected ':'
./stacks/shared-vpc.yml:38:21

The offending line is:
- { Key: prx:meta:tagging-version, Value: "2021-04-07" }

CloudFormation has no issues with the code (it creates those tags as expected). Quoting the values makes the linter happy, even in CodeBuild, but it shouldn't be necessary, and since the results are inconsistent between platforms it's hard/annoying to catch them during development. Also it's a false positive, so it's mainly just noise during CI.

@PatMyron
Copy link
Contributor

Interesting, wasn't able to reproduce on Ubuntu either. Wonder if it's specific to Amazon Linux

@kddejong
Copy link
Contributor

Just for reference we did find some oddities in Alpine systems too. #666 not sure if this is related or not but its worth noting we try to use the C parsers when available for speed reasons.

@kddejong
Copy link
Contributor

Possible references that we need to investigate. We need to start by knowing if it ends up using the Cparser or not and then what version of that library it may be using. yaml/libyaml#28 and yaml/pyyaml#45

@farski
Copy link
Author

farski commented Apr 27, 2021

Seems like the most recent release has resolved this

@PatMyron
Copy link
Contributor

maybe #1992?

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

No branches or pull requests

3 participants