Fix: use TSQLQuoter for column names escaping in bulkcopy

dlapko · dlapko · commit f5fd314be432 · 2025-04-21T18:01:43.000+03:00
diff --git a/bulkcopy.go b/bulkcopy.go
@@ -98,11 +98,12 @@ func (b *Bulk) sendBulkCommand(ctx context.Context) (err error) {
 
 	//columns definitions
 	var col_defs bytes.Buffer
+	q := TSQLQuoter{}
 	for i, col := range b.bulkColumns {
 		if i != 0 {
 			col_defs.WriteString(", ")
 		}
-		col_defs.WriteString("[" + col.ColName + "] " + makeDecl(col.ti))
+		col_defs.WriteString(q.ID(col.ColName) + " " + makeDecl(col.ti))
 	}
 
 	//options

Original file line number	Diff line number	Diff line change
`@@ -98,11 +98,12 @@ func (b *Bulk) sendBulkCommand(ctx context.Context) (err error) {`
`98`	`98`
`99`	`99`	`//columns definitions`
`100`	`100`	`var col_defs bytes.Buffer`
	`101`	`+ q := TSQLQuoter{}`
`101`	`102`	`for i, col := range b.bulkColumns {`
`102`	`103`	`if i != 0 {`
`103`	`104`	`col_defs.WriteString(", ")`
`104`	`105`	`}`
`105`		`- col_defs.WriteString("[" + col.ColName + "] " + makeDecl(col.ti))`
	`106`	`+ col_defs.WriteString(q.ID(col.ColName) + " " + makeDecl(col.ti))`
`106`	`107`	`}`
`107`	`108`
`108`	`109`	`//options`