Fix GoSec G104 linting errors

- Explicitly ignore errors for NewBufferString since they can't return an error
- Handle uncaught errors correctly

Author: boekkooi-fresh

driver/inmemory/internal/generate_matcher/main.go

@@ -2,6 +2,7 @@ package main
 
 import (
 	"flag"
+	"fmt"
 	"os"
 	"path/filepath"
 	"reflect"
@@ -200,7 +201,11 @@ func main() {
 	if err != nil {
 		panic(err)
 	}
-	defer f.Close()
+	defer func() {
+		if err := f.Close(); err != nil {
+			fmt.Printf("failed to close file: %s\n", err)
+		}
+	}()
 
 	err = tmpl.Execute(f, struct {
 		Types        []reflect.Kind

driver/inmemory/matcher.go

@@ -119,8 +119,8 @@ func (c *metadataConstraint) Matches(val interface{}) (bool, error) {
 func (e IncompatibleMatcherError) Error() string {
 	msg := bytes.NewBufferString("goengine: incompatible metadata.Matcher")
 	for _, err := range e {
-		msg.WriteRune('\n')
-		msg.WriteString(err.Error())
+		_, _ = msg.WriteRune('\n')
+		_, _ = msg.WriteString(err.Error())
 	}
 
 	return msg.String()

driver/sql/postgres/eventstore.go

@@ -254,17 +254,17 @@ func (e *EventStore) prepareInsertValues(streamEvents []goengine.Message, lenCol
 	for i := 0; i < placeholderCount; i++ {
 		if m := i % lenCols; m == 0 {
 			if i != 0 {
-				placeholders.WriteString("),")
+				_, _ = placeholders.WriteString("),")
 			}
-			placeholders.WriteRune('(')
+			_, _ = placeholders.WriteRune('(')
 		} else {
-			placeholders.WriteRune(',')
+			_, _ = placeholders.WriteRune(',')
 		}
 
-		placeholders.WriteRune('$')
-		placeholders.WriteString(strconv.Itoa(i + 1))
+		_, _ = placeholders.WriteRune('$')
+		_, _ = placeholders.WriteString(strconv.Itoa(i + 1))
 	}
-	placeholders.WriteString(")")
+	_, _ = placeholders.WriteString(")")
 	e.preparedInsertPlaceholder[messageCount] = placeholders.String()
 
 	return e.preparedInsertPlaceholder[messageCount]

test/internal/postgres.go

@@ -82,7 +82,7 @@ func (c *dbController) disableDatabaseAccess(t *testing.T, databaseName string)
 	// Terminate existing connections
 	rows, err := c.db.Query("SELECT pg_terminate_backend(pid) FROM pg_stat_activity WHERE datname = $1", databaseName)
 	require.NoError(t, err)
-	defer rows.Close()
+	require.NoError(t, err, rows.Close())
 }
 
 func (c *dbController) enableDatabaseAccess(t *testing.T, databaseName string) {