From 3c4d91a443bd5f0dc4d603d461b7b5cf6b4a34f1 Mon Sep 17 00:00:00 2001
From: sesheffield <sam@sheffinet.org>
Date: Wed, 7 Aug 2024 12:24:27 -0400
Subject: [PATCH] chore(linter): enable gosec on golangci linter and reformat
 config file (#1983)

add in gosec to the golangci.yml config file and reorder the linters-settings to be in alphabetical order
---
 .golangci.yml | 56 ++++++++++++++++++++++++++-------------------------
 1 file changed, 29 insertions(+), 27 deletions(-)

diff --git a/.golangci.yml b/.golangci.yml
index 65095e15..53b29e6b 100644
--- a/.golangci.yml
+++ b/.golangci.yml
@@ -14,71 +14,71 @@ linters:
     - contextcheck
     - decorder
     - dogsled
-#    - dupl
-#    - dupword
+    #    - dupl
+    #    - dupword
     - durationcheck
     - errcheck
     - errchkjson
     - errname
     - errorlint
-#    - exhaustive
+    #    - exhaustive
     - exportloopref
     - funlen
     - gci
     - ginkgolinter
     - gocheckcompilerdirectives
-#    - gochecknoglobals
-#    - gochecknoinits
+    #    - gochecknoglobals
+    #    - gochecknoinits
     - goconst
     - gocritic
     - godox
     - gofmt
-#    - gofumpt
+    #    - gofumpt
     - goheader
     - goimports
     - mnd
-#    - gomodguard
+    #    - gomodguard
     - goprintffuncname
-#    - gosec
+    - gosec
     - gosimple
     - govet
     - grouper
     - importas
     - ineffassign
-#    - interfacebloat
+    #    - interfacebloat
     - lll
     - loggercheck
     - makezero
     - mirror
     - misspell
     - musttag
-#    - nakedret
-#    - nestif
+    #    - nakedret
+    #    - nestif
     - nilerr
-#    - nilnil
-#    - noctx
+    #    - nilnil
+    #    - noctx
     - nolintlint
-#    - nonamedreturns
+    #    - nonamedreturns
     - nosprintfhostport
     - prealloc
     - predeclared
     - promlinter
-#    - reassign
+    #    - reassign
     - revive
     - rowserrcheck
     - staticcheck
-#    - stylecheck
+    #    - stylecheck
     - tagalign
-#    - testpackage
-#    - thelper
-#    - tparallel
+    #    - testpackage
+    #    - thelper
+    #    - tparallel
     - typecheck
-#    - unconvert
+    #    - unconvert
     - unparam
     - unused
-#    - usestdlibvars
+    #    - usestdlibvars
     - wastedassign
-#    - whitespace
+    #    - whitespace
     - wrapcheck
 
 
@@ -104,18 +104,20 @@ linters-settings:
       - BUG
       - FIXME
       - HACK
+  gosec:
+    exclude-generated: true
   lll:
     line-length: 120
+  misspell:
+    locale: US
+    ignore-words: expect
   nolintlint:
     allow-leading-space: false
     require-explanation: true
     require-specific: true
-  unparam:
-    check-exported: true # checks exported functions and methods for unused params
-  misspell:
-    locale: US
-    ignore-words: expect
   prealloc:
     simple: true # enables simple preallocation checks
     range-loops: true # enabled preallocation checks in range loops
     for-loops: false # disables preallocation checks in for loops
+  unparam:
+    check-exported: true # checks exported functions and methods for unused params