fix linting issues

Author: topi314

internal/httprate/counter.go

@@ -50,11 +50,9 @@ func (c *counter) Try(key string) (bool, int, time.Time) {
 func (c *counter) Cleanup() {
 	ticker := time.NewTicker(time.Second * 10)
 	defer ticker.Stop()
-	for {
-		select {
-		case <-ticker.C:
-			c.doCleanup()
-		}
+
+	for range ticker.C {
+		c.doCleanup()
 	}
 }
 
@@ -72,6 +70,6 @@ func (c *counter) doCleanup() {
 
 func counterKey(key string) uint64 {
 	h := xxhash.New()
-	h.WriteString(key)
+	_, _ = h.WriteString(key)
 	return h.Sum64()
 }

internal/httprate/limiter.go

@@ -77,7 +77,6 @@ func canonicalizeIP(ip string) string {
 		case ':':
 			// IPv6
 			isIPv6 = true
-			break
 		}
 	}
 	if !isIPv6 {

main.go

@@ -5,7 +5,6 @@ import (
 	"embed"
 	"flag"
 	"io/fs"
-	"log"
 	"log/slog"
 	"net/http"
 	"os"
@@ -107,9 +106,6 @@ func main() {
 		driver = sqlite.New
 	}
 
-	entries, err := Migrations.ReadDir(".")
-	log.Printf("entries: %v", entries)
-
 	if err = gomigrate.Migrate(ctx, db, driver, Migrations, gomigrate.WithDirectory("server/migrations")); err != nil {
 		slog.Error("Error while migrating database", tint.Err(err))
 		os.Exit(1)

server/database/db.go

@@ -18,7 +18,6 @@ import (
 	"github.com/topi314/gomigrate"
 	"go.opentelemetry.io/otel/attribute"
 	"go.opentelemetry.io/otel/semconv/v1.25.0"
-	"go.opentelemetry.io/otel/trace"
 	_ "modernc.org/sqlite"
 
 	"github.com/topi314/gobin/v2/internal/timex"
@@ -55,8 +54,8 @@ func (c Config) String() string {
 	str := fmt.Sprintf("\n  Type: %s\n  Debug: %t\n  ExpireAfter: %s\n  CleanupInterval: %s\n  ",
 		c.Type,
 		c.Debug,
-		c.ExpireAfter,
-		c.CleanupInterval,
+		time.Duration(c.ExpireAfter),
+		time.Duration(c.CleanupInterval),
 	)
 	switch c.Type {
 	case TypePostgres:
@@ -156,8 +155,7 @@ func New(ctx context.Context, cfg Config) (*DB, error) {
 
 type DB struct {
 	*sqlx.DB
-	rand   *rand.Rand
-	tracer trace.Tracer
+	rand *rand.Rand
 }
 
 func (d *DB) randomString(length int) string {

server/database/documents.go

@@ -159,7 +159,7 @@ func (d *DB) DeleteDocumentVersion(ctx context.Context, documentID string, docum
 	return &Document{
 		ID:      documentID,
 		Version: documentVersion,
-		Files:   files,
+		Files:   lastDeletedFiles,
 	}, nil
 }