Merge pull request #206 from karimra/set-json-ietf-enc

fix set cmd behavior with --encoding flag
openconfig · Oct 26, 2020 · 6aac40f · 6aac40f
2 parents 7512c91 + 1130f4f
commit 6aac40f
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/cmd/set.go b/cmd/set.go
@@ -316,7 +316,7 @@ func createSetRequest() (*gnmi.SetRequest, error) {
 				return nil, fmt.Errorf("encoding: %s not supported together with file values", viper.GetString("encoding"))
 			}
 		} else {
-			err = setValue(value, "json", setInput.updateValues[i])
+			err = setValue(value, strings.ToLower(viper.GetString("encoding")), setInput.updateValues[i])
 			if err != nil {
 				return nil, err
 			}
@@ -352,7 +352,7 @@ func createSetRequest() (*gnmi.SetRequest, error) {
 				return nil, fmt.Errorf("encoding: %s not supported together with file values", viper.GetString("encoding"))
 			}
 		} else {
-			err = setValue(value, "json", setInput.replaceValues[i])
+			err = setValue(value, strings.ToLower(viper.GetString("encoding")), setInput.replaceValues[i])
 			if err != nil {
 				return nil, err
 			}