fix: sensitive metadata suppression + missing tests + doc template fix - #1862
fix: sensitive metadata suppression + missing tests + doc template fix#1862Angel-4903 wants to merge 4 commits into
Conversation
This reverts commit 4add82c.
|
Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement Learn more about why HashiCorp requires a CLA and what the CLA includes 1 out of 2 committers have signed the CLA.
Angel Raphael seems not to be a GitHub user. Have you signed the CLA already but the status is still pending? Recheck it. |
1 similar comment
|
Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement Learn more about why HashiCorp requires a CLA and what the CLA includes 1 out of 2 committers have signed the CLA.
Angel Raphael seems not to be a GitHub user. Have you signed the CLA already but the status is still pending? Recheck it. |
Summary
Problems fixed
metadata.valuesemitted""whensuppress_metadata_valuesis enabled — not valid JSON, breaksjsondecode(). Fixed to emit"{}".stateMetadataNeedsSuppressiontreated"{}"as populated. Fixed by treating"{}"as the suppressed sentinel.docs/index.mdbut not intemplates/index.md.tmpl. Fixed.TestCloakSetValues*tests and addedTestDeepCloneMap,TestExtractSensitiveValues,TestStateMetadataNeedsSuppression_*.