Bump Go to 1.24/1.25

Author: nemith

.github/workflows/push.yaml

@@ -13,11 +13,11 @@ jobs:
       - name: checkout main branch
         uses: actions/checkout@v4
 
-      - uses: actions/setup-go@v4
+      - uses: actions/setup-go@v5
         with:
-          go-version: 1.22
+          go-version: '1.25'
 
-      - run: go test -coverprofile=cover.out ./...
+      - run: go test -v -race -coverprofile=cover.out ./...
 
       - name: checkout covrage branch
         uses: actions/checkout@v4

.github/workflows/validate.yaml

@@ -12,19 +12,25 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
-      - uses: extractions/setup-just@v2
-      - run: just lint
+      - uses: actions/setup-go@v5
+        with:
+          go-version: '1.25'
+      - uses: golangci/golangci-lint-action@v7
+        with:
+          version: v2.6.1
 
   test:
     name: Unit Tests
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        go: ["1.21", "1.22"]
+        go: ["1.24", "1.25"]
     steps:
       - uses: actions/checkout@v4
-      - uses: extractions/setup-just@v2
-      - run: GO_VERSION=${{ matrix.go }} just test
+      - uses: actions/setup-go@v5
+        with:
+          go-version: ${{ matrix.go }}
+      - run: go test -race ./...
 
   inttest:
     name: Integration Tests

.gitignore

@@ -1,3 +1,5 @@
 .idea/
 confd-basic*.zip
 out
+cover.out
+*.test

.golangci.yml

@@ -0,0 +1,26 @@
+run:
+  timeout: 10m0s
+
+linters:
+  enable:
+    - errcheck
+    - errorlint
+    - gofmt
+    - goimports
+    - govet
+    - ineffassign
+    - staticcheck
+    - unused
+
+linters-settings:
+  errcheck:
+    check-type-assertions: true
+    check-blank: true
+  
+  errorlint:
+    errorf: true
+    asserts: true
+    comparison: true
+
+issues:
+  exclude-use-default: false

Dockerfile

@@ -1,30 +1,6 @@
-ARG GO_VER=1.20
-ARG GOLANGCI_VER=1.51
+ARG GO_VER=1.24
 
-FROM golang:${GO_VER} as base
-WORKDIR /src
-COPY go.mod go.sum /src/
-RUN --mount=type=cache,target=/go/pkg/mod \
-    go mod download -x
-
-FROM golangci/golangci-lint:v${GOLANGCI_VER}-alpine AS lint-base
-
-FROM base AS lint
-RUN --mount=target=. \
-    --mount=from=lint-base,src=/usr/bin/golangci-lint,target=/usr/bin/golangci-lint \
-    --mount=type=cache,target=/go/pkg/mod \
-    --mount=type=cache,target=/root/.cache/go-build \
-    --mount=type=cache,target=/root/.cache/golangci-lint \
-    golangci-lint run --timeout 10m0s ./...
-
-FROM base as unittest
-RUN --mount=target=/src \
-    --mount=type=cache,target=/go/pkg/mod \
-    --mount=type=cache,target=/root/.cache/go-build \
-    mkdir /out && \
-    go test -v -race -coverprofile=/out/cover.out ./... | tee /out/test.stdout
-
-FROM base AS inttest
+FROM golang:${GO_VER} AS inttest
 RUN apt update && apt install -y \
     openssh-client \
     sshpass \
@@ -37,8 +13,4 @@ RUN --mount=target=/src \
     go test --tags=inttest -c -o /out/inttest.test
 WORKDIR /out
 COPY inttest/wait-for-hello.sh .
-CMD ./inttest.test -test.v -test.race
-
-
-FROM scratch AS unittest-coverage
-COPY --from=unittest /out /
+CMD ./inttest.test -test.v -test.race

README.md

@@ -8,7 +8,7 @@
 
 This library is used to create client applications for connecting to network devices via NETCONF.
 
-Like Go itself, only the latest two Go versions are tested and supported (Go 1.24 or Go 1.23).
+Like Go itself, only the latest two Go versions are tested and supported (Go 1.24 or Go 1.26).
 
 NOTICE: The API is pretty stable but not finalized yet. So changes may happen before a 1.0 release. Check for API changes when you upgrade.
 

go.mod

@@ -1,18 +1,16 @@
 module github.com/nemith/netconf
 
-go 1.23.0
-
-toolchain go1.24.1
+go 1.24.10
 
 require (
-	github.com/stretchr/testify v1.11.0
-	golang.org/x/crypto v0.34.0
+	github.com/stretchr/testify v1.11.1
+	golang.org/x/crypto v0.45.0
 )
 
 require (
 	github.com/davecgh/go-spew v1.1.1 // indirect
 	github.com/pmezard/go-difflib v1.0.0 // indirect
-	golang.org/x/exp v0.0.0-20230817173708-d852ddb80c63
-	golang.org/x/sys v0.30.0 // indirect
+	golang.org/x/exp v0.0.0-20251125195548-87e1e737ad39
+	golang.org/x/sys v0.38.0 // indirect
 	gopkg.in/yaml.v3 v3.0.1 // indirect
 )

go.sum

@@ -2,16 +2,16 @@ github.com/davecgh/go-spew v1.1.1 h1:vj9j/u1bqnvCEfJOwUhtlOARqs3+rkHYY13jYWTU97c
 github.com/davecgh/go-spew v1.1.1/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
 github.com/pmezard/go-difflib v1.0.0 h1:4DBwDE0NGyQoBHbLQYPwSUPoCMWR5BEzIk/f1lZbAQM=
 github.com/pmezard/go-difflib v1.0.0/go.mod h1:iKH77koFhYxTK1pcRnkKkqfTogsbg7gZNVY4sRDYZ/4=
-github.com/stretchr/testify v1.11.0 h1:ib4sjIrwZKxE5u/Japgo/7SJV3PvgjGiRNAvTVGqQl8=
-github.com/stretchr/testify v1.11.0/go.mod h1:wZwfW3scLgRK+23gO65QZefKpKQRnfz6sD981Nm4B6U=
-golang.org/x/crypto v0.34.0 h1:+/C6tk6rf/+t5DhUketUbD1aNGqiSX3j15Z6xuIDlBA=
-golang.org/x/crypto v0.34.0/go.mod h1:dy7dXNW32cAb/6/PRuTNsix8T+vJAqvuIy5Bli/x0YQ=
-golang.org/x/exp v0.0.0-20230817173708-d852ddb80c63 h1:m64FZMko/V45gv0bNmrNYoDEq8U5YUhetc9cBWKS1TQ=
-golang.org/x/exp v0.0.0-20230817173708-d852ddb80c63/go.mod h1:0v4NqG35kSWCMzLaMeX+IQrlSnVE/bqGSyC2cz/9Le8=
-golang.org/x/sys v0.30.0 h1:QjkSwP/36a20jFYWkSue1YwXzLmsV5Gfq7Eiy72C1uc=
-golang.org/x/sys v0.30.0/go.mod h1:/VUhepiaJMQUp4+oa/7Zr1D23ma6VTLIYjOOTFZPUcA=
-golang.org/x/term v0.29.0 h1:L6pJp37ocefwRRtYPKSWOWzOtWSxVajvz2ldH/xi3iU=
-golang.org/x/term v0.29.0/go.mod h1:6bl4lRlvVuDgSf3179VpIxBF0o10JUpXWOnI7nErv7s=
+github.com/stretchr/testify v1.11.1 h1:7s2iGBzp5EwR7/aIZr8ao5+dra3wiQyKjjFuvgVKu7U=
+github.com/stretchr/testify v1.11.1/go.mod h1:wZwfW3scLgRK+23gO65QZefKpKQRnfz6sD981Nm4B6U=
+golang.org/x/crypto v0.45.0 h1:jMBrvKuj23MTlT0bQEOBcAE0mjg8mK9RXFhRH6nyF3Q=
+golang.org/x/crypto v0.45.0/go.mod h1:XTGrrkGJve7CYK7J8PEww4aY7gM3qMCElcJQ8n8JdX4=
+golang.org/x/exp v0.0.0-20251125195548-87e1e737ad39 h1:DHNhtq3sNNzrvduZZIiFyXWOL9IWaDPHqTnLJp+rCBY=
+golang.org/x/exp v0.0.0-20251125195548-87e1e737ad39/go.mod h1:46edojNIoXTNOhySWIWdix628clX9ODXwPsQuG6hsK0=
+golang.org/x/sys v0.38.0 h1:3yZWxaJjBmCWXqhN1qh02AkOnCQ1poK6oF+a7xWL6Gc=
+golang.org/x/sys v0.38.0/go.mod h1:OgkHotnGiDImocRcuBABYBEXf8A9a87e/uXjp9XT3ks=
+golang.org/x/term v0.37.0 h1:8EGAD0qCmHYZg6J17DvsMy9/wJ7/D/4pV/wfnld5lTU=
+golang.org/x/term v0.37.0/go.mod h1:5pB4lxRNYYVZuTLmy8oR2BH8dflOR+IbTYFD8fi3254=
 gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405 h1:yhCVgyC4o1eVCa2tZl7eS0r+SDo693bJlVdllGtEeKM=
 gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405/go.mod h1:Co6ibVJAznAaIkqp8huTwlJQCZ016jof/cbN4VW5Yz0=
 gopkg.in/yaml.v3 v3.0.1 h1:fxVm/GzAzEWqLHuvctI91KS9hhNmmWOoWu0XTYJS7CA=

justfile

@@ -5,23 +5,11 @@ pwd := justfile_directory()
 @list:
     just --list
 
-golangci_version := "1.51"
-go_version := env_var_or_default("GO_VERSION", "1.20")
-
 test *args:
-    docker buildx build \
-        --build-arg GO_VER={{ go_version }} \
-        --output out/test \
-        --target unittest-coverage . \
-        {{ args }}
-    cat out/test/test.stdout
+    go test -v -race {{ args }}
 
 lint *args:
-    @docker buildx build \
-        --build-arg GO_VER={{ go_version }} \
-        --build-arg GOLANGCI_VER={{ golangci_version }} \
-        --target lint . \
-        {{ args }}
+    golangci-lint run --timeout 10m0s ./... {{ args }}
 
 inttest:
     just inttest/all

ops.go

@@ -322,7 +322,7 @@ type LockReq struct {
 
 func (s *Session) Lock(ctx context.Context, target Datastore) error {
 	req := LockReq{
-		XMLName: xml.Name{Local: "lock"},
+		XMLName: xml.Name{Space: "urn:ietf:params:xml:ns:netconf:base:1.0", Local: "lock"},
 		Target:  target,
 	}
 
@@ -332,7 +332,7 @@ func (s *Session) Lock(ctx context.Context, target Datastore) error {
 
 func (s *Session) Unlock(ctx context.Context, target Datastore) error {
 	req := LockReq{
-		XMLName: xml.Name{Local: "unlock"},
+		XMLName: xml.Name{Space: "urn:ietf:params:xml:ns:netconf:base:1.0", Local: "unlock"},
 		Target:  target,
 	}