From a82a3ccaa65d1551bc5c241b16dc506ec3fdaf87 Mon Sep 17 00:00:00 2001 From: Megan Marsh Date: Thu, 15 Aug 2019 16:16:11 -0700 Subject: [PATCH] need to scrub sensitive variables first, to make sure we don't fail for comma-containing sensitive info --- packer/ui.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/packer/ui.go b/packer/ui.go index 596c17253..d2f7bdca2 100644 --- a/packer/ui.go +++ b/packer/ui.go @@ -335,15 +335,15 @@ func (u *MachineReadableUi) Machine(category string, args ...string) { // Prepare the args for i, v := range args { - args[i] = strings.Replace(v, ",", "%!(PACKER_COMMA)", -1) - args[i] = strings.Replace(args[i], "\r", "\\r", -1) - args[i] = strings.Replace(args[i], "\n", "\\n", -1) // Use LogSecretFilter to scrub out sensitive variables for s := range LogSecretFilter.s { if s != "" { args[i] = strings.Replace(args[i], s, "", -1) } } + args[i] = strings.Replace(v, ",", "%!(PACKER_COMMA)", -1) + args[i] = strings.Replace(args[i], "\r", "\\r", -1) + args[i] = strings.Replace(args[i], "\n", "\\n", -1) } argsString := strings.Join(args, ",")