fix(plugin): check whether dataTemplate is an empty field (#1394)

* fix(plugin): check whether dataTemplate is an empty field

Signed-off-by: crui <crui@relaper.com>

* fix(plugin): check whether dataTemplate is a string before check empty

Signed-off-by: crui <crui@relaper.com>

Signed-off-by: crui <crui@relaper.com>
Co-authored-by: crui <crui@relaper.com>

extensions/sinks/influx/influx.go

@@ -44,12 +44,12 @@ type influxSink struct {
 	username     string
 	password     string
 	measurement  string
-	databasename string
-	tagkey       string
-	tagvalue     string
+	databaseName string
+	tagKey       string
+	tagValue     string
 	fields       string
 	cli          client.Client
-	fieldmap     map[string]interface{}
+	fieldMap     map[string]interface{}
 	hasTransform bool
 }
 
@@ -76,17 +76,17 @@ func (m *influxSink) Configure(props map[string]interface{}) error {
 	}
 	if i, ok := props["databasename"]; ok {
 		if i, ok := i.(string); ok {
-			m.databasename = i
+			m.databaseName = i
 		}
 	}
 	if i, ok := props["tagkey"]; ok {
 		if i, ok := i.(string); ok {
-			m.tagkey = i
+			m.tagKey = i
 		}
 	}
 	if i, ok := props["tagvalue"]; ok {
 		if i, ok := i.(string); ok {
-			m.tagvalue = i
+			m.tagValue = i
 		}
 	}
 	if i, ok := props["fields"]; ok {
@@ -94,8 +94,10 @@ func (m *influxSink) Configure(props map[string]interface{}) error {
 			m.fields = i
 		}
 	}
-	if _, ok := props["dataTemplate"]; ok {
-		m.hasTransform = true
+	if i, ok := props["dataTemplate"]; ok {
+		if i, ok := i.(string); ok && i != "" {
+			m.hasTransform = true
+		}
 	}
 	return nil
 }
@@ -143,23 +145,23 @@ func (m *influxSink) Collect(ctx api.StreamContext, data interface{}) error {
 	}
 
 	bp, err := client.NewBatchPoints(client.BatchPointsConfig{
-		Database:  m.databasename,
+		Database:  m.databaseName,
 		Precision: "ns",
 	})
 	if err != nil {
 		logger.Debug(err)
 		return err
 	}
-	tags := map[string]string{m.tagkey: m.tagvalue}
+	tags := map[string]string{m.tagKey: m.tagValue}
 	fields := strings.Split(m.fields, ",")
-	m.fieldmap = make(map[string]interface{}, 100)
+	m.fieldMap = make(map[string]interface{}, 100)
 	for _, field := range fields {
 		if output[field] != nil {
-			m.fieldmap[field] = output[field]
+			m.fieldMap[field] = output[field]
 		}
 	}
 
-	pt, err := client.NewPoint(m.measurement, tags, m.fieldmap, time.Now())
+	pt, err := client.NewPoint(m.measurement, tags, m.fieldMap, time.Now())
 	if err != nil {
 		logger.Debug(err)
 		return err