turbot / powerpipe

Powerpipe: Dashboards for DevOps. Visualize cloud configurations. Assess security posture against a massive library of benchmarks. Build custom dashboards with code.
https://powerpipe.io/
GNU Affero General Public License v3.0
228 stars 16 forks source link

.ppvars file parsing bug when defining variables that follows HCL formatting #302

Closed danlsgiga closed 4 months ago

danlsgiga commented 5 months ago

Describe the bug Defining a .ppvars config file with more than 1 spaces between the variable name and the = sign render the following error.

[ Error   ] failed to load variables from 'aws_tags.auto.ppvars'
Argument or block definition required: An argument or block definition is required here. To set an argument, use the equals sign "=" to introduce the argument value.
(aws_tags.auto.ppvars:3,1)

Powerpipe version (powerpipe -v) Example: v0.1.3

To reproduce

Expected behavior Since .ppvars configs are basically HCL, this should never happen since HCL accepts and, in fact, encourages this type of alignment via fmt

pskrbasu commented 4 months ago

@danlsgiga Thanks for raising this. We'll look into getting this fixed.