prepare release v3.33.2

Signed-off-by: Knut Ahlers <knut@ahlers.me>

.golangci.yml

@@ -31,11 +31,11 @@ linters:
     - bodyclose     # checks whether HTTP response body is closed successfully [fast: true, auto-fix: false]
     - containedctx  # containedctx is a linter that detects struct contained context.Context field [fast: true, auto-fix: false]
     - contextcheck  # check the function whether use a non-inherited context [fast: false, auto-fix: false]
+    - copyloopvar   # copyloopvar is a linter detects places where loop variables are copied [fast: true, auto-fix: false]
     - dogsled       # Checks assignments with too many blank identifiers (e.g. x, _, _, _, := f()) [fast: true, auto-fix: false]
     - durationcheck # check for two durations multiplied together [fast: false, auto-fix: false]
     - errcheck      # Errcheck is a program for checking for unchecked errors in go programs. These unchecked errors can be critical bugs in some cases [fast: false, auto-fix: false]
     - errchkjson    # Checks types passed to the json encoding functions. Reports unsupported types and optionally reports occations, where the check for the returned error can be omitted. [fast: false, auto-fix: false]
-    - exportloopref # checks for pointers to enclosing loop variables [fast: true, auto-fix: false]
     - forbidigo     # Forbids identifiers [fast: true, auto-fix: false]
     - funlen        # Tool for detection of long functions [fast: true, auto-fix: false]
     - gocognit      # Computes and checks the cognitive complexity of functions [fast: true, auto-fix: false]

History.md

@@ -1,3 +1,10 @@
+# 3.33.2 / 2024-08-27
+
+  * Bugfixes
+    * [overlays] Fix KoFi donation currency in eventfeed
+    * [raffle] Lint: Ignore linter false-positive
+    * [CI] Lint: Replace deprecated linter
+
 # 3.33.1 / 2024-08-14
 
   * Bugfixes

internal/apimodules/overlays/default/eventfeed.js

@@ -345,11 +345,11 @@ const app = createApp({
     handleKoFiDonation(eventId, data, time) {
       let text
       if (data.isSubscription && data.isFirstSubPayment) {
-        text = `${data.from} just started a monthly subscription of ${Number(data.amount).toFixed(2)}€`
+        text = `${data.from} just started a monthly subscription of ${Number(data.amount).toFixed(2)} ${data.currency}`
       } else if (data.isSubscription && !data.isFirstSubPayment) {
-        text = `${data.from} continued their monthly subscription of ${Number(data.amount).toFixed(2)}€`
+        text = `${data.from} continued their monthly subscription of ${Number(data.amount).toFixed(2)} ${data.currency}`
       } else {
-        text = `${data.from} just donated ${Number(data.amount).toFixed(2)}€`
+        text = `${data.from} just donated ${Number(data.amount).toFixed(2)} ${data.currency}`
       }
 
       this.addEvent({

internal/apimodules/raffle/pick.go

@@ -57,7 +57,7 @@ func (cryptRandSrc) Int63() int64 {
 		return -1
 	}
 	// mask off sign bit to ensure positive number
-	return int64(binary.LittleEndian.Uint64(b[:]) & (1<<63 - 1))
+	return int64(binary.LittleEndian.Uint64(b[:]) & (1<<63 - 1)) //#nosec:G115 - Masking ensures conversion is fine
 }
 
 // We're using a non-seedable source