Release/v1.13.0

.github/dependabot.yml

@@ -1,6 +1,20 @@
+---
 version: 2
 updates:
-  - package-ecosystem: "gomod" 
-    directory: "/" 
+  - package-ecosystem: "gomod"
+    open-pull-requests-limit: 10  # avoid spam, if no one reacts
+    directory: "/"
     schedule:
       interval: "weekly"
+
+  - package-ecosystem: "github-actions"
+    open-pull-requests-limit: 10  # avoid spam, if no one reacts
+    directory: "/"
+    schedule:
+      # Check for updates to GitHub Actions every week
+      interval: "weekly"
+    groups:
+      actions:
+        update-types:
+          - "minor"
+          - "patch"

.github/workflows/go.yml

@@ -1,3 +1,4 @@
+---
 name: Workflow-Pipeline
 
 on:
@@ -61,7 +62,7 @@ jobs:
           fetch-depth: 0
 
       - name: Set up Go ${{ matrix.go-version }}
-        uses: actions/setup-go@v4
+        uses: actions/setup-go@v5
         with:
           go-version: ${{ matrix.go-version }}
         id: Go
@@ -82,7 +83,7 @@ jobs:
 
       - name: Upload Test Coverage
         if: ${{ matrix.go-version == '1.22'}}
-        uses: actions/upload-artifact@v3
+        uses: actions/upload-artifact@v4
         with:
           name: Example-Test-Report
           path: profile.cov
@@ -101,7 +102,7 @@ jobs:
           fetch-depth: 0
 
       - name: Set up Go ${{ matrix.go-version }}
-        uses: actions/setup-go@v4
+        uses: actions/setup-go@v5
         with:
           go-version: ${{ matrix.go-version }}
         id: Go
@@ -119,7 +120,7 @@ jobs:
 
       - name: Upload Test Coverage
         if: ${{ matrix.go-version == '1.22'}}
-        uses: actions/upload-artifact@v3
+        uses: actions/upload-artifact@v4
         with:
           name: PKG-Coverage-Report
           path: profile.cov
@@ -133,7 +134,7 @@ jobs:
         uses: actions/checkout@v4
 
       - name: Download Coverage Report
-        uses: actions/download-artifact@v3
+        uses: actions/download-artifact@v4
         with:
           path: artifacts
 
@@ -167,7 +168,7 @@ jobs:
         uses: actions/checkout@v4
 
       - name: Download Coverage Report
-        uses: actions/download-artifact@v3
+        uses: actions/download-artifact@v4
         with:
           path: artifacts
 
@@ -178,7 +179,7 @@ jobs:
           tail -n +2 ./PKG-Coverage-Report/profile.cov >> merged_profile.cov
 
       - name: Upload
-        uses: paambaati/codeclimate-action@v5.0.0
+        uses: paambaati/codeclimate-action@v8.0.0
         env:
           CC_TEST_REPORTER_ID: ${{ secrets.CC_TEST_REPORTER_ID }}
         with:
@@ -188,7 +189,7 @@ jobs:
   code_quality:
     name: Code Quality🎖️
     runs-on: ubuntu-latest
-    container: "golangci/golangci-lint:v1.57.2"
+    container: "golangci/golangci-lint:v1.59.1"
     steps:
       - name: Check out code into the Go module directory
         uses: actions/checkout@v4
@@ -197,3 +198,14 @@ jobs:
       - name: GolangCI-Lint
         run: |
           golangci-lint run --timeout 9m0s
+
+  linting_party:
+    name: Linting Party🥳
+    runs-on: ubuntu-latest
+    steps:
+      - name: Check out code
+        uses: actions/checkout@v4
+      - name: Check for file names errors
+        uses: ls-lint/action@v2.2.3
+        with:
+          config: .ls-lint.yml

.github/workflows/website_prod.yml

@@ -24,10 +24,10 @@ jobs:
         uses: actions/checkout@v4
 
       - name: Set up Docker Buildx
-        uses: docker/setup-buildx-action@v2
+        uses: docker/setup-buildx-action@v3
 
       - name: Log in to the GitHub Container registry
-        uses: docker/login-action@v1
+        uses: docker/login-action@v3
         with:
           registry: ${{ env.WEBSITE_REGISTRY }}
           username: ${{ github.actor }}
@@ -38,7 +38,7 @@ jobs:
           docker pull ${{ env.WEBSITE_REGISTRY }}/gofr-dev/website:latest
 
       - name: Build Image
-        uses: docker/build-push-action@v4
+        uses: docker/build-push-action@v6
         with:
           push: false
           context: ./
@@ -56,17 +56,17 @@ jobs:
         uses: actions/checkout@v4
 
       - name: Set up Docker Buildx
-        uses: docker/setup-buildx-action@v2
+        uses: docker/setup-buildx-action@v3
 
       - name: Login to GAR
-        uses: docker/login-action@v2
+        uses: docker/login-action@v3
         with:
           registry: us-central1-docker.pkg.dev
           username: _json_key
           password: ${{ secrets.deploy_key }}
 
       - name: Log in to the GitHub Container registry
-        uses: docker/login-action@v1
+        uses: docker/login-action@v3
         with:
           registry: ${{ env.WEBSITE_REGISTRY }}
           username: ${{ github.actor }}
@@ -77,7 +77,7 @@ jobs:
           docker pull ${{ env.WEBSITE_REGISTRY }}/gofr-dev/website:latest
 
       - name: Build and Push Image Stage
-        uses: docker/build-push-action@v4
+        uses: docker/build-push-action@v6
         with:
           push: true
           context: ./
@@ -102,7 +102,7 @@ jobs:
         uses: actions/checkout@v4
 
       - name: Authorize to GCP service account
-        uses: google-github-actions/auth@v1
+        uses: google-github-actions/auth@v2
         with:
           credentials_json: ${{ secrets.deploy_key }}
 

.golangci.yml

@@ -1,3 +1,4 @@
+---
 linters-settings:
   dupl:
     threshold: 100
@@ -8,7 +9,9 @@ linters-settings:
     statements: 50
   gci:
     sections:
-      - prefix(gofr.dev)
+      - standard
+      - default
+      - localmodule
   goconst:
     min-len: 2
     min-occurrences: 2
@@ -31,7 +34,7 @@ linters-settings:
     local-prefixes: github.com/golangci/golangci-lint
   golint:
     min-confidence: 0
-  gomnd:
+  mnd:
     checks:
       - argument
       - case
@@ -55,6 +58,36 @@ linters-settings:
     allow-unused: false # report any unused nolint directives
     require-explanation: true # require an explanation for nolint directives
     require-specific: true # require nolint directives to be specific about which linter is being skipped
+  revive:
+    rules:
+    # default revive rules, they have to be present otherwise they are disabled
+      - name: blank-imports
+      - name: context-as-argument
+      - name: context-keys-type
+      - name: dot-imports
+      - name: empty-block
+      - name: error-naming
+      - name: error-return
+      - name: error-strings
+      - name: errorf
+      - name: increment-decrement
+      - name: indent-error-flow
+      - name: range
+      - name: receiver-naming
+      - name: redefines-builtin-id
+      - name: superfluous-else
+      - name: time-naming
+      - name: unexported-return
+      - name: unreachable-code
+      - name: unused-parameter
+      - name: var-declaration
+      - name: var-naming
+    # additional revive rules
+      - name: bare-return
+      - name: bool-literal-in-expr
+      - name: comment-spacings
+      - name: early-return
+      - name: defer
 
 linters:
   # please, do not use `enable-all`: it's deprecated and will be removed soon.
@@ -65,7 +98,9 @@ linters:
     - bodyclose
     - dogsled
     - dupl
+    - err113
     - errcheck
+    - errorlint
     - exhaustive
     - exportloopref
     - funlen
@@ -76,17 +111,17 @@ linters:
     - gocritic
     - gocyclo
     - godot
-    - goerr113
     - gofmt
     - goimports
-    - gomnd
     - goprintffuncname
     - gosec
     - gosimple
     - govet
     - ineffassign
     - lll
+    - mirror
     - misspell
+    - mnd
     - nakedret
     - nestif
     - noctx
@@ -96,9 +131,11 @@ linters:
     - rowserrcheck
     - staticcheck
     - stylecheck
+    - thelper
     - unconvert
     - unparam
     - unused
+    - usestdlibvars
     - whitespace
     - wsl
 
@@ -108,7 +145,7 @@ linters:
 
 
 service:
-  golangci-lint-version: 1.57.x
+  golangci-lint-version: 1.59.x
 
 issues:
   # exclude-use-default: false

.ls-lint.yml

@@ -0,0 +1,4 @@
+---
+ls:
+  .go: snake_case
+  .pb.go: snake_case

docs/references/configs/page.md

@@ -300,4 +300,9 @@ This document lists all the configuration options supported by the GoFr framewor
 -  MQTT_QOS
 -  Quality of Service Level
 
+---
+
+-  MQTT_KEEP_ALIVE
+-  Sends regular messages to check the link is active. May not work as expected if handling func is blocking execution
+
 {% /table %}

examples/http-server/main_test.go

@@ -38,7 +38,7 @@ func TestIntegration_SimpleAPIServer(t *testing.T) {
 	}
 
 	for i, tc := range tests {
-		req, _ := http.NewRequest("GET", host+tc.path, nil)
+		req, _ := http.NewRequest(http.MethodGet, host+tc.path, nil)
 		req.Header.Set("content-type", "application/json")
 
 		c := http.Client{}
@@ -92,7 +92,7 @@ func TestIntegration_SimpleAPIServer_Errors(t *testing.T) {
 	}
 
 	for i, tc := range tests {
-		req, _ := http.NewRequest("GET", host+tc.path, nil)
+		req, _ := http.NewRequest(http.MethodGet, host+tc.path, nil)
 		req.Header.Set("content-type", "application/json")
 
 		c := http.Client{}
@@ -129,7 +129,7 @@ func TestIntegration_SimpleAPIServer_Health(t *testing.T) {
 	}
 
 	for i, tc := range tests {
-		req, _ := http.NewRequest("GET", host+tc.path, nil)
+		req, _ := http.NewRequest(http.MethodGet, host+tc.path, nil)
 		req.Header.Set("content-type", "application/json")
 
 		c := http.Client{}

examples/using-custom-metrics/main_test.go

@@ -16,22 +16,22 @@ func TestIntegration(t *testing.T) {
 
 	c := http.Client{}
 
-	req, _ := http.NewRequest("POST", host+"/transaction", nil)
+	req, _ := http.NewRequest(http.MethodPost, host+"/transaction", nil)
 	req.Header.Set("content-type", "application/json")
 
 	_, err := c.Do(req)
 	if err != nil {
 		t.Fatalf("request to /transaction failed %v", err)
 	}
 
-	req, _ = http.NewRequest("POST", host+"/return", nil)
+	req, _ = http.NewRequest(http.MethodPost, host+"/return", nil)
 
 	_, err = c.Do(req)
 	if err != nil {
 		t.Fatalf("request to /transaction failed %v", err)
 	}
 
-	req, _ = http.NewRequest("GET", "http://localhost:2120/metrics", nil)
+	req, _ = http.NewRequest(http.MethodGet, "http://localhost:2120/metrics", nil)
 
 	resp, err := c.Do(req)
 	if err != nil {

examples/using-file-bind/main_test.go

@@ -2,13 +2,14 @@ package main
 
 import (
 	"bytes"
-	"github.com/stretchr/testify/assert"
 	"io"
 	"mime/multipart"
 	"net/http"
 	"os"
 	"testing"
 	"time"
+
+	"github.com/stretchr/testify/assert"
 )
 
 func TestMain_BindError(t *testing.T) {