ci: multiple updates

- bump Go toolchain to 1.25
- update actions versions
- replace gokart with gosec
- simplify format and lint checks
- replace golint with revive

Author: anitgandhi

.github/workflows/go.yml

@@ -11,55 +11,60 @@ jobs:
   build:
     runs-on: ubuntu-latest
     steps:
-    - uses: actions/checkout@v2
-
     - name: Set up Go
-      uses: actions/setup-go@v2
+      uses: actions/setup-go@v6
       with:
-        go-version: 1.17
+        go-version: 1.25
 
-    - name: OVS setup
+    - name: Set up OVS
       run: |
        sudo apt-get update
         sudo apt install openvswitch-switch
         sudo ovs-vsctl add-br ovsbr0
-        
-    - name: Check and test
-      run: |
-        export GOPATH=/home/runner/work
-        export PATH=$PATH:$GOPATH/bin
-        mkdir $GOPATH/src $GOPATH/pkg $GOPATH/bin
-        go install honnef.co/go/tools/cmd/staticcheck@2020.2.1
-        NEW=$GOPATH/src/github.com/digitalocean/go-openvswitch
-        mkdir -p $NEW
-        cp -r ./* $NEW
-        cd $NEW
-        go mod download
-        go get golang.org/x/lint/golint
-        go get -d ./...
+
+    - uses: actions/checkout@v5
+
+    - name: License check
+      run: |     
         echo "=========START LICENSE CHECK============"
         ./scripts/licensecheck.sh
-        echo "=========START BUILD============"
-        go build -v -tags=gofuzz ./...
-        echo "=========START VET============"
+
+    - name: Download deps
+      run: go mod download
+  
+    - name: Lint
+      run: |
+        # install tools
+        go install honnef.co/go/tools/cmd/staticcheck@latest
+        go install golang.org/x/lint/golint@latest
+        go install github.com/mgechev/revive@latest
+
         go vet ./...
-        echo "=========START GO FMT CHECK============"
-        ./scripts/gofmt.sh
-        echo "=========START GO LINT CHECK============"
-        ./scripts/golint.sh
-        echo "=========START STATICCHECK============"
+
+        GOFMT=$(gofmt -l .)
+        if [[ -n $GOFMT ]]; then
+          echo -e "Files that are not gofmt'd:\n"
+          echo "$GOFMT"
+          exit 1
+        fi
+
+        revive ./...
+        
         staticcheck ./...
-        echo "=========START LINT============"
-        golint -set_exit_status ./cmd/... ./internal/...
+
+    - name: Run Gosec Security Scanner
+      uses: securego/gosec@master
+      with:
+        args: -exclude=G115 ./...
+
+    - name: Build
+      run: go build -v ./...
+        
+    - name: Test
+      run: |
         echo "=========START TESTS IN OVS============"
         go test -v -race -short ./ovs/
         echo "=========START TESTS IN OVSDB============"
-        go test -v -race ./ovsdb/
-        go test -c -race ./ovsdb
+        sudo go test -v -race ./ovsdb/
         echo "=========START TESTS IN OVSNL============"
         go test -v -race ./ovsnl/
-        echo "=========START OVSDB.TEST============"
-        sudo ./ovsdb.test -test.v
-        echo "=========START SECURITY SCANNING============"
-        go install github.com/praetorian-inc/gokart@latest
-        gokart scan

revive.toml

@@ -0,0 +1,31 @@
+ignoreGeneratedHeader = true
+severity = "warning"
+confidence = 0.8
+errorCode = 1
+warningCode = 1
+exclude = ["ovsnl/internal/ovsh"]
+
+[rule.blank-imports]
+[rule.context-as-argument]
+[rule.context-keys-type]
+[rule.dot-imports]
+[rule.error-return]
+[rule.error-strings]
+[rule.error-naming]
+[rule.exported]
+[rule.increment-decrement]
+[rule.var-naming]
+[rule.var-declaration]
+[rule.package-comments]
+[rule.range]
+[rule.receiver-naming]
+[rule.time-naming]
+[rule.unexported-return]
+[rule.indent-error-flow]
+[rule.errorf]
+[rule.empty-block]
+[rule.superfluous-else]
+[rule.unused-parameter]
+    Disabled = true
+[rule.unreachable-code]
+[rule.redefines-builtin-id]