Skip to content

Commit

Permalink
Merge pull request #58 from Juniper/fix_pre_gnmi_regression
Browse files Browse the repository at this point in the history
remove extra escape if present and fixed regression for internal jtimon pre-gnmi logging
  • Loading branch information
amohit1315 authored Oct 18, 2024
2 parents d6fccff + 4a3c218 commit 98b4e55
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions internal_jtimon.go
Original file line number Diff line number Diff line change
Expand Up @@ -227,7 +227,7 @@ func jLogInternalJtimonForGnmi(jctx *JCtx, parseOutput *gnmiParseOutputT, rsp *g
keyVal := strings.Split(result[1], ":")
s += fmt.Sprintf("\t\tval {\n\t\t\t%s: %s\n\t\t}\n", keyVal[0], keyVal[1])
update["key"] = keyVal[0]
update["value"] = keyVal[1]
update["value"] = strings.Trim(keyVal[1], "\"")
}

updates = append(updates, update)
Expand Down Expand Up @@ -255,7 +255,7 @@ func jLogInternalJtimonForGnmi(jctx *JCtx, parseOutput *gnmiParseOutputT, rsp *g
}

func jLogInternalJtimonForPreGnmi(jctx *JCtx, ocdata *na_pb.OpenConfigData, outString string) {
if jctx.config.InternalJtimon.logger == nil || jctx.config.InternalJtimon.DataLog != "" {
if jctx.config.InternalJtimon.logger == nil || jctx.config.InternalJtimon.DataLog == "" {
return
}

Expand Down

0 comments on commit 98b4e55

Please sign in to comment.