Merge pull request #6 from AcidSailor/main

karimra · web-flow · commit bad7879fb589 · 2022-11-08T06:32:52.000-08:00
fix(outputs): AddSubscriptionTarget correct return with empty addTarget
diff --git a/outputs/output.go b/outputs/output.go
@@ -81,6 +81,9 @@ func DecodeConfig(src, dst interface{}) error {
 
 func AddSubscriptionTarget(msg proto.Message, meta Meta, addTarget string, tpl *template.Template) (*gnmi.SubscribeResponse, error) {
 	if addTarget == "" {
+		if message, ok := msg.(*gnmi.SubscribeResponse); ok {
+			return message, nil
+		}
 		return nil, nil
 	}
 	msg = proto.Clone(msg)

Original file line number	Diff line number	Diff line change
`@@ -81,6 +81,9 @@ func DecodeConfig(src, dst interface{}) error {`
`81`	`81`
`82`	`82`	`func AddSubscriptionTarget(msg proto.Message, meta Meta, addTarget string, tpl template.Template) (gnmi.SubscribeResponse, error) {`
`83`	`83`	`if addTarget == "" {`
	`84`	`+ if message, ok := msg.(*gnmi.SubscribeResponse); ok {`
	`85`	`+ return message, nil`
	`86`	`+ }`
`84`	`87`	`return nil, nil`
`85`	`88`	`}`
`86`	`89`	`msg = proto.Clone(msg)`