chore(deps): update docker.io/library/golang docker tag to v1.26

Co-authored-by: Renovate Bot <renovate@t00n.de>
Co-committed-by: Renovate Bot <renovate@t00n.de>

.gitea/workflows/cd.yaml

@@ -4,6 +4,14 @@ on:
   push:
     branches:
       - main
+    paths:
+      - "go.mod"
+      - "go.sum"
+      - "**/*.go"
+      - "config.example.yaml"
+      - "Dockerfile"
+      - "Makefile"
+  workflow_dispatch:
 
 env:
   DOCKER_REGISTRY: gitea.t000-n.de
@@ -11,17 +19,19 @@ env:
 jobs:
   test:
     name: test
-    runs-on: ubuntu-latest
+    runs-on:
+      - ubuntu-latest
+      - linux_amd64
     steps:
       - name: Checkout
-        uses: actions/checkout@v4
+        uses: actions/checkout@9c091bb21b7c1c1d1991bb908d89e4e9dddfe3e0 # v7.0.0
       - name: Setup go
-        uses: actions/setup-go@v5
+        uses: actions/setup-go@924ae3a1cded613372ab5595356fb5720e22ba16 # v6.5.0
         with:
           go-version-file: go.mod
           check-latest: true
       - name: Create cache key
-        uses: https://gitea.com/actions/go-hashfiles@v0.0.1
+        uses: https://gitea.com/actions/go-hashfiles@264ae76b7e50173ce71ed7da4b48e5e517f3f9ec # v0.0.1
         id: hash-go
         with:
           patterns: |
@@ -29,7 +39,7 @@ jobs:
             go.sum
       - name: cache go
         id: cache-go
-        uses: actions/cache@v4
+        uses: actions/cache@27d5ce7f107fe9357f9df03efb73ab90386fccae # v5
         with:
           path: |
             /go_path
@@ -44,25 +54,72 @@ jobs:
 
   build_and_push:
     name: Build and push
+    strategy:
+      matrix:
+        arch: [ amd64, arm64 ]
     needs:
       - test
-    runs-on: ubuntu-latest
+    runs-on:
+      - ubuntu-latest
+      - linux_${{ matrix.arch }}
     steps:
       - name: Checkout
-        uses: actions/checkout@v4
+        uses: actions/checkout@9c091bb21b7c1c1d1991bb908d89e4e9dddfe3e0 # v7.0.0
-
-      - name: Set up QEMU
-        uses: docker/setup-qemu-action@v2
-
       - name: Set up Docker Buildx
-        uses: docker/setup-buildx-action@v2
+        uses: docker/setup-buildx-action@bb05f3f5519dd87d3ba754cc423b652a5edd6d2c # v4.2.0
-
       - name: Login to Registry
-        uses: docker/login-action@v2
+        uses: docker/login-action@af1e73f918a031802d376d3c8bbc3fe56130a9b0 # v4.4.0
         with:
           registry: ${{ env.DOCKER_REGISTRY }}
           username: ${{ secrets.REGISTRY_USER }}
           password: ${{ secrets.REGISTRY_PASSWORD }}
+      - name: Get Metadata
+        id: meta
+        run: |
+          echo REPO_NAME=$(echo ${GITHUB_REPOSITORY} | awk -F"/" '{print $2}' | tr '[:upper:]' '[:lower:]') >> $GITHUB_OUTPUT
+          echo REPO_VERSION=$(git describe --tags --always | sed 's/^v//') >> $GITHUB_OUTPUT
+      - name: Build and push
+        uses: docker/build-push-action@53b7df96c91f9c12dcc8a07bcb9ccacbed38856a # v7.3.0
+        with:
+          context: .
+          file: ./Dockerfile
+          platforms: linux/${{ matrix.arch }}
+          push: true
+          provenance: false
+          build-args: GOARCH=${{ matrix.arch }}
+          tags: |
+            ${{ env.DOCKER_REGISTRY }}/t.behrendt/${{ steps.meta.outputs.REPO_NAME }}:${{ steps.meta.outputs.REPO_VERSION }}-${{ matrix.arch }}
+
+  create_tag:
+    name: Create tag
+    runs-on: ubuntu-latest
+    outputs:
+      tag: ${{ steps.tag.outputs.new-tag }}
+    steps:
+      - uses: actions/checkout@9c091bb21b7c1c1d1991bb908d89e4e9dddfe3e0 # v7.0.0
+        with:
+          fetch-depth: 0
+      - uses: https://gitea.t000-n.de/t.behrendt/conventional-semantic-git-tag-increment@e393dbb067c09fd43519392572ccf63b66cba92d # 0.1.39
+        id: tag
+        with:
+          token: ${{ secrets.GITEA_TOKEN }}
+          prerelease: ${{ github.event_name == 'workflow_dispatch' }}
+      - run: |
+          git tag ${{ steps.tag.outputs.new-tag }}
+          git push origin ${{ steps.tag.outputs.new-tag }}
+      - name: Set output
+        run: |
+          echo "tag=${{ steps.tag.outputs.new-tag }}" >> $GITHUB_OUTPUT
+
+  create_manifest:
+    name: Create manifest
+    needs:
+      - build_and_push
+      - create_tag
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@9c091bb21b7c1c1d1991bb908d89e4e9dddfe3e0 # v7.0.0
 
       - name: Get Metadata
         id: meta
@@ -70,15 +127,17 @@ jobs:
           echo REPO_NAME=$(echo ${GITHUB_REPOSITORY} | awk -F"/" '{print $2}' | tr '[:upper:]' '[:lower:]') >> $GITHUB_OUTPUT
           echo REPO_VERSION=$(git describe --tags --always | sed 's/^v//') >> $GITHUB_OUTPUT
 
-      - name: Build and push
+      - name: Login to Registry
-        uses: docker/build-push-action@v4
+        uses: docker/login-action@af1e73f918a031802d376d3c8bbc3fe56130a9b0 # v4.4.0
         with:
-          context: .
+          registry: ${{ env.DOCKER_REGISTRY }}
-          file: ./Dockerfile
+          username: ${{ secrets.REGISTRY_USER }}
-          platforms: |
+          password: ${{ secrets.REGISTRY_PASSWORD }}
-            linux/amd64
+
-            linux/arm64
+      - name: Create manifest
-          push: true
+        run: |
-          tags: |
+          docker manifest create ${{ env.DOCKER_REGISTRY }}/t.behrendt/${{ steps.meta.outputs.REPO_NAME }}:${{ needs.create_tag.outputs.tag }} \
-            ${{ env.DOCKER_REGISTRY }}/t.behrendt/${{ steps.meta.outputs.REPO_NAME }}:${{ steps.meta.outputs.REPO_VERSION }}
+            ${{ env.DOCKER_REGISTRY }}/t.behrendt/${{ steps.meta.outputs.REPO_NAME }}:${{ steps.meta.outputs.REPO_VERSION }}-amd64 \
-            ${{ env.DOCKER_REGISTRY }}/t.behrendt/${{ steps.meta.outputs.REPO_NAME }}:latest
+            ${{ env.DOCKER_REGISTRY }}/t.behrendt/${{ steps.meta.outputs.REPO_NAME }}:${{ steps.meta.outputs.REPO_VERSION }}-arm64
+
+          docker manifest push ${{ env.DOCKER_REGISTRY }}/t.behrendt/${{ steps.meta.outputs.REPO_NAME }}:${{ needs.create_tag.outputs.tag }}

.gitea/workflows/ci.yaml

@@ -10,17 +10,19 @@ env:
 jobs:
   test:
     name: test
-    runs-on: ubuntu-latest
+    runs-on:
+      - ubuntu-latest
+      - linux_amd64
     steps:
       - name: Checkout
-        uses: actions/checkout@v4
+        uses: actions/checkout@9c091bb21b7c1c1d1991bb908d89e4e9dddfe3e0 # v7.0.0
       - name: Setup go
-        uses: actions/setup-go@v5
+        uses: actions/setup-go@924ae3a1cded613372ab5595356fb5720e22ba16 # v6.5.0
         with:
           go-version-file: go.mod
           check-latest: true
       - name: Create cache key
-        uses: https://gitea.com/actions/go-hashfiles@v0.0.1
+        uses: https://gitea.com/actions/go-hashfiles@264ae76b7e50173ce71ed7da4b48e5e517f3f9ec # v0.0.1
         id: hash-go
         with:
           patterns: |
@@ -28,7 +30,7 @@ jobs:
             go.sum
       - name: cache go
         id: cache-go
-        uses: actions/cache@v4
+        uses: actions/cache@27d5ce7f107fe9357f9df03efb73ab90386fccae # v5
         with:
           path: |
             /go_path
@@ -42,3 +44,10 @@ jobs:
         run: make test
       - name: check:format
         run: make check-format
+      - name: Upload test-coverage
+        uses: ChristopherHX/gitea-upload-artifact@v4
+        with:
+          name: realdnydns-test-coverage
+          path: |
+            lcov.info
+          retention-days: 1

.gitignore

@@ -23,3 +23,5 @@ go.work
 
 .vscode
 config.yaml
+lcov.info
+realdnydns

Dockerfile

@@ -1,13 +1,14 @@
-FROM golang:1.23-alpine
+FROM docker.io/library/golang:1.26-alpine@sha256:9097beb5536220f7857bdcb65c1b4b340630dd7a70b85f03d5af29640b06693d as build
+
+ARG GOARCH=amd64
 
 WORKDIR /app
-
 COPY go.mod go.sum ./
-
 RUN go mod download
-
 COPY . .
+RUN CGO_ENABLED=0 GOOS=linux GOARCH=${GOARCH} \
+    go build -trimpath -ldflags="-s -w" -o main .
 
-RUN go build -o main .
+FROM gcr.io/distroless/static-debian12@sha256:9c346e4be81b5ca7ff31a0d89eaeade58b0f95cfd3baed1f36083ddb47ca3160
-
+COPY --from=build /app/main /
-CMD ["./main"]
+CMD ["/main"]

Makefile

@@ -1,11 +1,19 @@
-test:
+test: test-unit test-race test-coverage
+
+test-unit:
 	go test ./pkg/... -coverprofile=coverage.out
 
+test-race:
+	go test ./pkg/... -race	
+
+test-coverage:
+	go tool gcov2lcov -infile coverage.out > lcov.info
+
 build:
 	go build
 
 lint:
-	golint ./...
+	go tool golint ./...
 
 run:
 	make build

config.example.yaml

@@ -8,6 +8,10 @@ dns_provider:
   config:
     api_key: <your-api-key>
     base_url: https://api.hosting.ionos.com/dns
+    # Optional: default TTL to use when patching records with missing TTL (default: 300)
+    # default_ttl: 300
+    # Optional: default priority to use when patching records with missing priority (default: 0)
+    # default_prio: 0
 notification_provider:
   type: gotify
   config:

go.mod

@@ -1,15 +1,26 @@
 module realdnydns
 
-go 1.23
+go 1.25.0
 
 require (
+	gitea.t000-n.de/t.behrendt/ionosDnsClient v1.0.2
 	github.com/go-co-op/gocron v1.37.0
 	gopkg.in/yaml.v3 v3.0.1
 )
 
 require (
+	github.com/apapsch/go-jsonmerge/v2 v2.0.0 // indirect
 	github.com/google/uuid v1.6.0 // indirect
+	github.com/jandelgado/gcov2lcov v1.1.1 // indirect
+	github.com/oapi-codegen/runtime v1.1.1 // indirect
 	github.com/robfig/cron/v3 v3.0.1 // indirect
-	github.com/stretchr/testify v1.8.4 // indirect
+	github.com/stretchr/testify v1.11.1 // indirect
 	go.uber.org/atomic v1.11.0 // indirect
+	golang.org/x/lint v0.0.0-20241112194109-818c5a804067 // indirect
+	golang.org/x/tools v0.0.0-20200130002326-2f3ba24bd6e7 // indirect
+)
+
+tool (
+	github.com/jandelgado/gcov2lcov
+	golang.org/x/lint/golint
 )

go.sum

@@ -1,3 +1,9 @@
+gitea.t000-n.de/t.behrendt/ionosDnsClient v1.0.2 h1:EWz4kLLv7lSZx/F8K0/WxN5xeVNh7z4qbeDurZYBcNk=
+gitea.t000-n.de/t.behrendt/ionosDnsClient v1.0.2/go.mod h1:HZfdMF7X9LK/3FP1jJVzMWbM5BcgcZg/Qwx2WClFipg=
+github.com/RaveNoX/go-jsoncommentstrip v1.0.0/go.mod h1:78ihd09MekBnJnxpICcwzCMzGrKSKYe4AqU6PDYYpjk=
+github.com/apapsch/go-jsonmerge/v2 v2.0.0 h1:axGnT1gRIfimI7gJifB699GoE/oq+F2MU7Dml6nw9rQ=
+github.com/apapsch/go-jsonmerge/v2 v2.0.0/go.mod h1:lvDnEdqiQrp0O42VQGgmlKpxL1AP2+08jFMw88y4klk=
+github.com/bmatcuk/doublestar v1.1.1/go.mod h1:UD6OnuiIn0yFxxA2le/rnRU1G4RaI4UvFv1sNto9p6w=
 github.com/creack/pty v1.1.9/go.mod h1:oKZEueFk5CKHvIhNR5MUki03XCEU+Q6VDXinZuGJ33E=
 github.com/davecgh/go-spew v1.1.0/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
 github.com/davecgh/go-spew v1.1.1 h1:vj9j/u1bqnvCEfJOwUhtlOARqs3+rkHYY13jYWTU97c=
@@ -7,6 +13,9 @@ github.com/go-co-op/gocron v1.37.0/go.mod h1:3L/n6BkO7ABj+TrfSVXLRzsP26zmikL4ISk
 github.com/google/uuid v1.4.0/go.mod h1:TIyPZe4MgqvfeYDBFedMoGGpEw/LqOeaOT+nhxU+yHo=
 github.com/google/uuid v1.6.0 h1:NIvaJDMOsjHA8n1jAhLSgzrAzy1Hgr+hNrb57e+94F0=
 github.com/google/uuid v1.6.0/go.mod h1:TIyPZe4MgqvfeYDBFedMoGGpEw/LqOeaOT+nhxU+yHo=
+github.com/jandelgado/gcov2lcov v1.1.1 h1:CHUNoAglvb34DqmMoZchnzDbA3yjpzT8EoUvVqcAY+s=
+github.com/jandelgado/gcov2lcov v1.1.1/go.mod h1:tMVUlMVtS1po2SB8UkADWhOT5Y5Q13XOce2AYU69JuI=
+github.com/juju/gnuflag v0.0.0-20171113085948-2ce1bb71843d/go.mod h1:2PavIy+JPciBPrBUjwbNvtwB6RQlve+hkpll6QSNmOE=
 github.com/kr/pretty v0.1.0/go.mod h1:dAy3ld7l9f0ibDNOQOHHMYYIIbhfbHSm3C4ZsoJORNo=
 github.com/kr/pretty v0.2.1/go.mod h1:ipq/a2n7PKx3OHsz4KJII5eveXtPO4qwEXGdVfWzfnI=
 github.com/kr/pretty v0.3.0 h1:WgNl7dwNpEZ6jJ9k1snq4pZsg7DOEN8hP9Xw0Tsjwk0=
@@ -15,6 +24,8 @@ github.com/kr/pty v1.1.1/go.mod h1:pFQYn66WHrOpPYNljwOMqo10TkYh1fy3cYio2l3bCsQ=
 github.com/kr/text v0.1.0/go.mod h1:4Jbv+DJW3UT/LiOwJeYQe1efqtUx/iVham/4vfdArNI=
 github.com/kr/text v0.2.0 h1:5Nx0Ya0ZqY2ygV366QzturHI13Jq95ApcVaJBhpS+AY=
 github.com/kr/text v0.2.0/go.mod h1:eLer722TekiGuMkidMxC/pM04lWEeraHUUmBw8l2grE=
+github.com/oapi-codegen/runtime v1.1.1 h1:EXLHh0DXIJnWhdRPN2w4MXAzFyE4CskzhNLUmtpMYro=
+github.com/oapi-codegen/runtime v1.1.1/go.mod h1:SK9X900oXmPWilYR5/WKPzt3Kqxn/uS/+lbpREv+eCg=
 github.com/pkg/diff v0.0.0-20210226163009-20ebb0f2a09e/go.mod h1:pJLUxLENpZxwdsKMEsNbx1VGcRFpLqf3715MtcvvzbA=
 github.com/pmezard/go-difflib v1.0.0 h1:4DBwDE0NGyQoBHbLQYPwSUPoCMWR5BEzIk/f1lZbAQM=
 github.com/pmezard/go-difflib v1.0.0/go.mod h1:iKH77koFhYxTK1pcRnkKkqfTogsbg7gZNVY4sRDYZ/4=
@@ -23,18 +34,36 @@ github.com/robfig/cron/v3 v3.0.1/go.mod h1:eQICP3HwyT7UooqI/z+Ov+PtYAWygg1TEWWzG
 github.com/rogpeppe/go-internal v1.6.1/go.mod h1:xXDCJY+GAPziupqXw64V24skbSoqbTEfhy4qGm1nDQc=
 github.com/rogpeppe/go-internal v1.8.1 h1:geMPLpDpQOgVyCg5z5GoRwLHepNdb71NXb67XFkP+Eg=
 github.com/rogpeppe/go-internal v1.8.1/go.mod h1:JeRgkft04UBgHMgCIwADu4Pn6Mtm5d4nPKWu0nJ5d+o=
+github.com/spkg/bom v0.0.0-20160624110644-59b7046e48ad/go.mod h1:qLr4V1qq6nMqFKkMo8ZTx3f+BZEkzsRUY10Xsm2mwU0=
 github.com/stretchr/objx v0.1.0/go.mod h1:HFkY916IF+rwdDfMAkV7OtwuqBVzrE8GR6GFx+wExME=
 github.com/stretchr/objx v0.4.0/go.mod h1:YvHI0jy2hoMjB+UWwv71VJQ9isScKT/TqJzVSSt89Yw=
 github.com/stretchr/objx v0.5.0/go.mod h1:Yh+to48EsGEfYuaHDzXPcE3xhTkx73EhmCGUpEOglKo=
+github.com/stretchr/objx v0.5.2/go.mod h1:FRsXN1f5AsAjCGJKqEizvkpNtU+EGNCLh3NxZ/8L+MA=
 github.com/stretchr/testify v1.3.0/go.mod h1:M5WIy9Dh21IEIfnGCwXGc5bZfKNJtfHm1UVUgZn+9EI=
 github.com/stretchr/testify v1.7.1/go.mod h1:6Fq8oRcR53rry900zMqJjRRixrwX3KX962/h/Wwjteg=
 github.com/stretchr/testify v1.8.0/go.mod h1:yNjHg4UonilssWZ8iaSj1OCr/vHnekPRkoO+kdMU+MU=
 github.com/stretchr/testify v1.8.2/go.mod h1:w2LPCIKwWwSfY2zedu0+kehJoqGctiVI29o6fzry7u4=
-github.com/stretchr/testify v1.8.4 h1:CcVxjf3Q8PM0mHUKJCdn+eZZtm5yQwehR5yeSVQQcUk=
 github.com/stretchr/testify v1.8.4/go.mod h1:sz/lmYIOXD/1dqDmKjjqLyZ2RngseejIcXlSw2iwfAo=
+github.com/stretchr/testify v1.9.0/go.mod h1:r2ic/lqez/lEtzL7wO/rwa5dbSLXVDPFyf8C91i36aY=
+github.com/stretchr/testify v1.11.1 h1:7s2iGBzp5EwR7/aIZr8ao5+dra3wiQyKjjFuvgVKu7U=
+github.com/stretchr/testify v1.11.1/go.mod h1:wZwfW3scLgRK+23gO65QZefKpKQRnfz6sD981Nm4B6U=
 go.uber.org/atomic v1.9.0/go.mod h1:fEN4uk6kAWBTFdckzkM89CLk9XfWZrxpCo0nPH17wJc=
 go.uber.org/atomic v1.11.0 h1:ZvwS0R+56ePWxUNi+Atn9dWONBPp/AUETXlHW0DxSjE=
 go.uber.org/atomic v1.11.0/go.mod h1:LUxbIzbOniOlMKjJjyPfpl4v+PKK2cNJn91OQbhoJI0=
+golang.org/x/crypto v0.0.0-20190308221718-c2843e01d9a2/go.mod h1:djNgcEr1/C05ACkg1iLfiJU5Ep61QUkGW8qpdssI0+w=
+golang.org/x/crypto v0.0.0-20191011191535-87dc89f01550/go.mod h1:yigFU9vqHzYiE8UmvKecakEJjdnWj3jj499lnFckfCI=
+golang.org/x/lint v0.0.0-20241112194109-818c5a804067 h1:adDmSQyFTCiv19j015EGKJBoaa7ElV0Q1Wovb/4G7NA=
+golang.org/x/lint v0.0.0-20241112194109-818c5a804067/go.mod h1:3xt1FjdF8hUf6vQPIChWIBhFzV8gjjsPE/fR3IyQdNY=
+golang.org/x/mod v0.1.1-0.20191105210325-c90efee705ee/go.mod h1:QqPTAvyqsEbceGzBzNggFXnrqF1CaUcvgkdR5Ot7KZg=
+golang.org/x/net v0.0.0-20190404232315-eb5bcb51f2a3/go.mod h1:t9HGtf8HONx5eT2rtn7q6eTqICYqUVnKs3thJo3Qplg=
+golang.org/x/net v0.0.0-20190620200207-3b0461eec859/go.mod h1:z5CRVTTTmAJ677TzLLGU+0bjPO0LkuOLi4/5GtJWs/s=
+golang.org/x/sync v0.0.0-20190423024810-112230192c58/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM=
+golang.org/x/sys v0.0.0-20190215142949-d0b11bdaac8a/go.mod h1:STP8DvDyc/dI5b8T5hshtkjS+E42TnysNCUPdjciGhY=
+golang.org/x/sys v0.0.0-20190412213103-97732733099d/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
+golang.org/x/text v0.3.0/go.mod h1:NqM8EUOU14njkJ3fqMW+pc6Ldnwhi/IjpwHt7yyuwOQ=
+golang.org/x/tools v0.0.0-20200130002326-2f3ba24bd6e7 h1:EBZoQjiKKPaLbPrbpssUfuHtwM6KV/vb4U85g/cigFY=
+golang.org/x/tools v0.0.0-20200130002326-2f3ba24bd6e7/go.mod h1:TB2adYChydJhpapKDTa4BR/hXlZSLoq2Wpct/0txZ28=
+golang.org/x/xerrors v0.0.0-20191011141410-1b5146add898/go.mod h1:I/5z698sn9Ka8TeJc9MKroUUfqBBauWjQqLJ2OPfmY0=
 gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405/go.mod h1:Co6ibVJAznAaIkqp8huTwlJQCZ016jof/cbN4VW5Yz0=
 gopkg.in/check.v1 v1.0.0-20180628173108-788fd7840127/go.mod h1:Co6ibVJAznAaIkqp8huTwlJQCZ016jof/cbN4VW5Yz0=
 gopkg.in/check.v1 v1.0.0-20201130134442-10cb98267c6c h1:Hei/4ADfdWqJk1ZMxUNpqntNwaWcugrBjAiHlqqRiVk=

pkg/dnsProvider/ionos/api/ionosAPI.go

@@ -1,224 +0,0 @@
-package ionosAPI
-
-import (
-	"bytes"
-	"encoding/json"
-	"errors"
-	"io"
-	"net"
-	"net/http"
-	"net/url"
-	"realdnydns/model/common"
-)
-
-/**
- * Docs: https://developer.hosting.ionos.com/docs/dns
- */
-type IonosAPI interface {
-	GetARecord(tld string, subdomain string) (*common.ARecord, error)
-	SetARecord(tld string, subdomain string, ip net.IP, ttl int, prio int, disabled bool) (*common.ARecord, error)
-	GetZoneId(tld string) (string, error)
-	GetRecordId(zoneId string, tld string, subdomain string, recordType string) (string, error)
-	HttpCall(method string, url string, body io.Reader, queryParams map[string]string) (*http.Response, error)
-}
-
-type IonosAPIImpl struct {
-	APIKey     string
-	BaseURL    string
-	HTTPClient *http.Client
-}
-
-type ZonesResponse []struct {
-	Name string `json:"name"`
-	Id   string `json:"id"`
-	Type string `json:"type"`
-}
-
-type ZoneResponse struct {
-	Id      string           `json:"id"`
-	Name    string           `json:"name"`
-	Type    string           `json:"type"`
-	Records []RecordResponse `json:"records"`
-}
-
-type RecordResponse struct {
-	Id         string `json:"id"`
-	Name       string `json:"name"`
-	RootName   string `json:"rootName"`
-	Type       string `json:"type"`
-	Content    string `json:"content"`
-	ChangeDate string `json:"changeDate"`
-	TTL        int    `json:"ttl"`
-	Prio       int    `json:"prio"`
-	Disabled   bool   `json:"disabled"`
-}
-
-type ChangeRecord struct {
-	Name     string `json:"name"`
-	Type     string `json:"type"`
-	Content  string `json:"content"`
-	TTL      int    `json:"ttl"`
-	Prio     int    `json:"prio"`
-	Disabled bool   `json:"disabled"`
-}
-
-type ChangeRecordRequest struct {
-	Content  string `json:"content"`
-	TTL      int    `json:"ttl"`
-	Prio     int    `json:"prio"`
-	Disabled bool   `json:"disabled"`
-}
-
-func New(APIKey string, BaseURL string) IonosAPI {
-	return &IonosAPIImpl{
-		APIKey:     APIKey,
-		BaseURL:    BaseURL,
-		HTTPClient: &http.Client{},
-	}
-}
-
-func (i *IonosAPIImpl) HttpCall(method string, path string, body io.Reader, queryParams map[string]string) (*http.Response, error) {
-	requestUrl, _ := url.Parse(i.BaseURL + path)
-
-	query := requestUrl.Query()
-	for key, value := range queryParams {
-		query.Add(key, value)
-	}
-	requestUrl.RawQuery = query.Encode()
-
-	req, err := http.NewRequest(method, requestUrl.String(), body)
-	if err != nil {
-		return nil, err
-	}
-
-	req.Header.Add("X-API-Key", i.APIKey)
-	req.Header.Add("Content-Type", "application/json")
-	req.Header.Add("Accept", "application/json")
-
-	return i.HTTPClient.Do(req)
-}
-
-func (i *IonosAPIImpl) GetZoneId(tld string) (string, error) {
-	res, err := i.HttpCall("GET", "/v1/zones", nil, nil)
-	if err != nil {
-		return "", err
-	}
-
-	responseBody := make([]byte, res.ContentLength)
-	res.Body.Read(responseBody)
-
-	zones := []ZoneResponse{}
-	json.Unmarshal(responseBody, &zones)
-
-	for _, z := range zones {
-		if z.Name == tld {
-			return z.Id, nil
-		}
-	}
-
-	return "", errors.New("zone not found")
-}
-
-func (i *IonosAPIImpl) GetRecordId(zoneId string, tld string, subdomain string, recordType string) (string, error) {
-	var domain string
-	if subdomain == "@" || subdomain == "" {
-		domain = tld
-	} else {
-		domain = subdomain + "." + tld
-	}
-
-	res, err := i.HttpCall("GET", "/v1/zones/"+zoneId, nil, map[string]string{"recordName": domain, "recordType": recordType})
-	if err != nil {
-		return "", err
-	}
-
-	responseBody := make([]byte, res.ContentLength)
-	res.Body.Read(responseBody)
-
-	zone := ZoneResponse{}
-	json.Unmarshal(responseBody, &zone)
-
-	for _, record := range zone.Records {
-		if record.Type == recordType && record.Name == domain {
-			return record.Id, nil
-		}
-	}
-
-	return "", errors.New("record not found")
-}
-
-func (i *IonosAPIImpl) SetARecord(tld string, subdomain string, ip net.IP, ttl int, prio int, disabled bool) (*common.ARecord, error) {
-	zoneId, err := i.GetZoneId(tld)
-	if err != nil {
-		return nil, err
-	}
-
-	recordId, err := i.GetRecordId(zoneId, tld, subdomain, "A")
-	if err != nil {
-		return nil, err
-	}
-
-	changeRecordRequest, err := json.Marshal(ChangeRecordRequest{
-		Content:  ip.String(),
-		TTL:      ttl,
-		Prio:     prio,
-		Disabled: false,
-	})
-	if err != nil {
-		return nil, err
-	}
-
-	res, err := i.HttpCall("PUT", "/v1/zones/"+zoneId+"/records/"+recordId, bytes.NewReader(changeRecordRequest), nil)
-	if err != nil {
-		return nil, err
-	}
-
-	responseBody := make([]byte, res.ContentLength)
-	res.Body.Read(responseBody)
-
-	if res.StatusCode != 200 {
-		return nil, errors.New("error updating record")
-	}
-
-	changeRecord := ChangeRecord{}
-	json.Unmarshal(responseBody, &changeRecord)
-
-	return &common.ARecord{
-		Domain:   changeRecord.Name,
-		IP:       changeRecord.Content,
-		TTL:      changeRecord.TTL,
-		Prio:     changeRecord.Prio,
-		Disabled: changeRecord.Disabled,
-	}, nil
-}
-
-func (ionos *IonosAPIImpl) GetARecord(tld string, subdomain string) (*common.ARecord, error) {
-	zoneId, err := ionos.GetZoneId(tld)
-	if err != nil {
-		return nil, err
-	}
-
-	recordId, err := ionos.GetRecordId(zoneId, tld, subdomain, "A")
-	if err != nil {
-		return nil, err
-	}
-
-	res, err := ionos.HttpCall("GET", "/v1/zones/"+zoneId+"/records/"+recordId, nil, nil)
-	if err != nil {
-		return nil, err
-	}
-
-	responseBody := make([]byte, res.ContentLength)
-	res.Body.Read(responseBody)
-
-	record := RecordResponse{}
-	json.Unmarshal(responseBody, &record)
-
-	return &common.ARecord{
-		Domain:   record.Name,
-		IP:       record.Content,
-		TTL:      record.TTL,
-		Prio:     record.Prio,
-		Disabled: record.Disabled,
-	}, nil
-}

pkg/dnsProvider/ionos/api/ionosAPI_test.go

@@ -1,285 +0,0 @@
-package ionosAPI
-
-import (
-	"encoding/json"
-	"net"
-	"net/http"
-	"net/http/httptest"
-	"testing"
-)
-
-func utilMockServerImpl() func(w http.ResponseWriter, r *http.Request) {
-	zonesResponse := []ZoneResponse{
-		{
-			Name: "example.com",
-			Id:   "1234567890",
-			Type: "NATIVE",
-		},
-		{
-			Name: "notTheExample.org",
-			Id:   "0987654321",
-			Type: "SLAVE",
-		},
-	}
-	zonesResponseJson, _ := json.Marshal(zonesResponse)
-
-	recordResponseSub := RecordResponse{
-		Id:         "abcdefghij",
-		Name:       "example.com",
-		RootName:   "example.com",
-		Type:       "A",
-		Content:    "127.0.0.1",
-		ChangeDate: "2019-12-09T13:04:25.772Z",
-		TTL:        300,
-		Prio:       0,
-		Disabled:   false,
-	}
-	recordResponseSubJson, _ := json.Marshal(recordResponseSub)
-
-	recordResponseTLD := RecordResponse{
-		Id:         "jihgfedcba",
-		Name:       "sub.example.com",
-		RootName:   "example.com",
-		Type:       "A",
-		Content:    "127.0.0.2",
-		ChangeDate: "2019-12-09T13:04:25.772Z",
-		TTL:        300,
-		Prio:       0,
-		Disabled:   false,
-	}
-	recordResponseTLDJson, _ := json.Marshal(recordResponseTLD)
-
-	zoneResponse := ZoneResponse{
-		Id:   "1234567890",
-		Name: "example.com",
-		Type: "NATIVE",
-		Records: []RecordResponse{
-			recordResponseSub,
-			recordResponseTLD,
-		},
-	}
-	zoneResponseJson, _ := json.Marshal(zoneResponse)
-
-	changeRecord := ChangeRecord{
-		Name:     "sub.example.com",
-		Type:     "A",
-		Content:  "127.0.0.1",
-		TTL:      300,
-		Prio:     0,
-		Disabled: false,
-	}
-	changeRecordJson, _ := json.Marshal(changeRecord)
-
-	return func(w http.ResponseWriter, r *http.Request) {
-		var response []byte
-
-		if r.Method == "GET" {
-			if r.RequestURI == "/v1/zones" {
-				response = zonesResponseJson
-			} else if r.RequestURI == "/v1/zones/1234567890?recordName=example.com&recordType=A" ||
-				r.RequestURI == "/v1/zones/1234567890?recordName=sub.example.com&recordType=A" {
-				response = zoneResponseJson
-			} else if r.RequestURI == "/v1/zones/1234567890/records/abcdefghij" {
-				response = recordResponseSubJson
-			} else if r.RequestURI == "/v1/zones/1234567890/records/jihgfedcba" {
-				response = recordResponseTLDJson
-			}
-		} else if r.Method == "PUT" {
-			response = changeRecordJson
-		} else {
-			response = []byte("404")
-		}
-
-		w.Write(response)
-	}
-}
-
-func TestHttpCall(t *testing.T) {
-	mockServer := httptest.NewServer(http.HandlerFunc(utilMockServerImpl()))
-	defer mockServer.Close()
-
-	ionosAPI := New("dummyKey", mockServer.URL)
-
-	t.Run("returns response for GET request", testHttpCallGet(ionosAPI))
-	t.Run("returns response for PUT request", testHttpCallPut(ionosAPI))
-	t.Run("returns error for non existing endpoint", testHttpCallNonExistingEndpoint())
-}
-
-func testHttpCallGet(api IonosAPI) func(t *testing.T) {
-	return func(t *testing.T) {
-		res, err := api.HttpCall("GET", "/v1/zones", nil, nil)
-
-		if err != nil {
-			t.Fatalf("HttpCall() returned unexpected error: %v", err)
-		}
-
-		if res.StatusCode != 200 {
-			t.Fatalf("HttpCall() returned unexpected status code: %v instead of 200", res.StatusCode)
-		}
-	}
-}
-
-func testHttpCallPut(api IonosAPI) func(t *testing.T) {
-	return func(t *testing.T) {
-		res, err := api.HttpCall("PUT", "/v1/zones/1234567890/records/abcdefghij", nil, nil)
-
-		if err != nil {
-			t.Fatalf("HttpCall() returned unexpected error: %v", err)
-		}
-
-		if res.StatusCode != 200 {
-			t.Fatalf("HttpCall() returned unexpected status code: %v instead of 200", res.StatusCode)
-		}
-	}
-}
-
-func testHttpCallNonExistingEndpoint() func(t *testing.T) {
-	return func(t *testing.T) {
-		mockServer := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-			w.WriteHeader(404)
-		}))
-		defer mockServer.Close()
-
-		api := New("dummyKey", mockServer.URL)
-
-		res, err := api.HttpCall("GET", "/v1/nonExistingEndpoint", nil, nil)
-
-		if err != nil {
-			t.Fatalf("HttpCall() returned unexpected error: %v", err)
-		}
-
-		if res.StatusCode != 404 {
-			t.Fatalf("HttpCall() returned unexpected status code: %v instead of 404", res.StatusCode)
-		}
-	}
-}
-
-func TestGetZoneId(t *testing.T) {
-	mockServer := httptest.NewServer(http.HandlerFunc(utilMockServerImpl()))
-
-	ionosAPI := New("dummyKey", mockServer.URL)
-
-	t.Run("returns zoneId for tracked domain", testGetZoneIdSuccess(ionosAPI))
-	t.Run("returns error for non tracked domain", testGetZoneIdNoMatch(ionosAPI))
-}
-
-func testGetZoneIdSuccess(api IonosAPI) func(t *testing.T) {
-	return func(t *testing.T) {
-		zoneId, err := api.GetZoneId("example.com")
-
-		if err != nil {
-			t.Fatalf("GetZoneId() returned unexpected error: %v", err)
-		}
-
-		if zoneId != "1234567890" {
-			t.Fatalf("GetZoneId() returned unexpected zoneId: %v instead of 1234567890", zoneId)
-		}
-	}
-}
-
-func testGetZoneIdNoMatch(api IonosAPI) func(t *testing.T) {
-	return func(t *testing.T) {
-		zoneId, err := api.GetZoneId("nonTrackedDomain.com")
-
-		if err == nil || zoneId != "" {
-			t.Fatalf("GetZoneId() did not return an error for a non tracked domain")
-		}
-	}
-}
-
-func TestGetRecordId(t *testing.T) {
-	mockServer := httptest.NewServer(http.HandlerFunc(utilMockServerImpl()))
-	defer mockServer.Close()
-
-	ionosAPI := New("dummyKey", mockServer.URL)
-
-	t.Run("returns zoneId for top level domain", testGetRecordIdForTopLevelDomain(ionosAPI))
-	t.Run("returns zoneId for subdomain", testGetRecordIdForSubdomain(ionosAPI))
-	t.Run("returns error for untracked subdomain", testGetRecordIdWithUntrackedSubdomain(ionosAPI))
-}
-
-func testGetRecordIdForTopLevelDomain(api IonosAPI) func(t *testing.T) {
-	return func(t *testing.T) {
-		recordId, err := api.GetRecordId("1234567890", "example.com", "", "A")
-
-		if err != nil {
-			t.Fatalf("GetZoneId() returned unexpected error: %v", err)
-		}
-
-		if recordId != "abcdefghij" {
-			t.Fatalf("GetZoneId() returned unexpected zoneId: %v instead of abcdefghij", recordId)
-		}
-	}
-}
-
-func testGetRecordIdForSubdomain(api IonosAPI) func(t *testing.T) {
-	return func(t *testing.T) {
-		recordId, err := api.GetRecordId("1234567890", "example.com", "sub", "A")
-
-		if err != nil {
-			t.Fatalf("GetZoneId() returned unexpected error: %v", err)
-		}
-
-		if recordId != "jihgfedcba" {
-			t.Fatalf("GetZoneId() returned unexpected zoneId: %v instead of jihgfedcba", recordId)
-		}
-	}
-}
-
-func testGetRecordIdWithUntrackedSubdomain(api IonosAPI) func(t *testing.T) {
-	return func(t *testing.T) {
-		recordId, err := api.GetRecordId("1234567890", "example.com", "untrackedSub", "A")
-
-		if err == nil && recordId == "" {
-			t.Fatalf("GetZoneId() did not return an error for a non tracked domain")
-		}
-	}
-}
-
-func TestGetARecord(t *testing.T) {
-	mockServer := httptest.NewServer(http.HandlerFunc(utilMockServerImpl()))
-	defer mockServer.Close()
-
-	ionosAPI := New("dummyKey", mockServer.URL)
-
-	t.Run("returns A record for top level domain", testGetARecordFor(ionosAPI, "example.com", ""))
-	t.Run("returns A record for subdomain", testGetARecordFor(ionosAPI, "example.com", "sub"))
-}
-
-func testGetARecordFor(api IonosAPI, tld string, subdomain string) func(t *testing.T) {
-	return func(t *testing.T) {
-		var domain string
-		if subdomain == "" {
-			domain = tld
-		} else {
-			domain = subdomain + "." + tld
-		}
-
-		record, err := api.GetARecord(tld, subdomain)
-
-		if err != nil {
-			t.Fatalf("GetARecord() returned unexpected error: %v", err)
-		}
-
-		if record.Domain != domain {
-			t.Fatalf("GetARecord() returned unexpected record for: %v instead of %v", record.Domain, domain)
-		}
-	}
-}
-
-func TestSetARecord(t *testing.T) {
-	mockServer := httptest.NewServer(http.HandlerFunc(utilMockServerImpl()))
-	defer mockServer.Close()
-
-	ionosAPI := New("dummyKey", mockServer.URL)
-
-	changedARecord, err := ionosAPI.SetARecord("example.com", "sub", net.ParseIP("127.0.0.1"), 300, 0, false)
-
-	if err != nil {
-		t.Fatalf("SetARecord() returned unexpected error: %v", err)
-	}
-
-	if changedARecord.Domain != "sub.example.com" {
-		t.Fatalf("SetARecord() returned unexpected record for: %v instead of sub.example.com", changedARecord.Domain)
-	}
-}

pkg/dnsProvider/ionos/ionos.go

@@ -1,20 +1,30 @@
 package ionosDnsProvider
 
 import (
+	"context"
 	"errors"
 	"net"
+	"net/http"
 	"realdnydns/model/common"
 	"realdnydns/pkg/dnsProvider"
-	ionosAPI "realdnydns/pkg/dnsProvider/ionos/api"
+	"sync"
+
+	ionosDnsClient "gitea.t000-n.de/t.behrendt/ionosDnsClient"
 )
 
 type IONOS struct {
-	API ionosAPI.IonosAPI
+	client         *ionosDnsClient.ClientWithResponses
+	zoneIdMap      map[string]string
+	defaultTtl     int
+	defaultPrio    int
+	zoneIdMapMutex sync.Mutex
 }
 
 type IONOSConfig struct {
 	APIKey      string `yaml:"api_key"`
 	BaseURL     string `yaml:"base_url"`
+	DefaultTTL  *int   `yaml:"default_ttl,omitempty"`
+	DefaultPrio *int   `yaml:"default_prio,omitempty"`
 }
 
 func NewIonos(config *IONOSConfig) (dnsProvider.DNSProvider, error) {
@@ -26,15 +36,231 @@ func NewIonos(config *IONOSConfig) (dnsProvider.DNSProvider, error) {
 		return nil, errors.New("base_url is required")
 	}
 
+	options := []ionosDnsClient.ClientOption{
+		ionosDnsClient.WithRequestEditorFn(func(ctx context.Context, req *http.Request) error {
+			req.Header.Set("X-API-Key", config.APIKey)
+			return nil
+		}),
+	}
+
+	client, err := ionosDnsClient.NewClientWithResponses(config.BaseURL, options...)
+	if err != nil {
+		return nil, err
+	}
+
+	// Set default values for TTL and Prio if not provided
+	defaultTtl := 300
+	if config.DefaultTTL != nil {
+		defaultTtl = *config.DefaultTTL
+	}
+
+	defaultPrio := 0
+	if config.DefaultPrio != nil {
+		defaultPrio = *config.DefaultPrio
+	}
+
 	return &IONOS{
-		ionosAPI.New(config.APIKey, config.BaseURL),
+		client:      client,
+		zoneIdMap:   make(map[string]string),
+		defaultTtl:  defaultTtl,
+		defaultPrio: defaultPrio,
 	}, nil
 }
 
-func (ionos *IONOS) UpdateRecord(tld string, subdomain string, ip net.IP, ttl int, prio int, disabled bool) (*common.ARecord, error) {
+func (i *IONOS) UpdateRecord(tld string, subdomain string, ip net.IP, ttl int, prio int, disabled bool) (*common.ARecord, error) {
-	return ionos.API.SetARecord(tld, subdomain, ip, ttl, prio, disabled)
+
+	zoneId, err := i.getZoneIdForTld(tld)
+	if err != nil {
+		return nil, err
 	}
 
-func (ionos *IONOS) GetRecord(tld string, subdomain string) (*common.ARecord, error) {
+	domain := assembleTldSubdomainToDomain(tld, subdomain)
-	return ionos.API.GetARecord(tld, subdomain)
+	recordId, err := i.getRecordIdForDomain(zoneId, domain)
+	if err != nil {
+		return nil, err
+	}
+
+	ipString := ip.String()
+	res, err := i.client.UpdateRecordWithResponse(context.Background(), zoneId, recordId, ionosDnsClient.RecordUpdate{
+		Content:  &ipString,
+		Ttl:      &ttl,
+		Prio:     &prio,
+		Disabled: &disabled,
+	})
+	if err != nil || res.StatusCode() != 200 {
+		return nil, errors.New("failed to update record")
+	}
+
+	if res.JSON200 == nil {
+		return nil, errors.New("record response is empty")
+	}
+
+	record := *res.JSON200
+	if record.Name == nil || record.Content == nil {
+		return nil, errors.New("record is empty")
+	}
+
+	// Handle optional fields with defaults
+	recordTtl := 0
+	if record.Ttl != nil {
+		recordTtl = *record.Ttl
+	}
+
+	recordPrio := 0
+	if record.Prio != nil {
+		recordPrio = *record.Prio
+	}
+
+	recordDisabled := false
+	if record.Disabled != nil {
+		recordDisabled = *record.Disabled
+	}
+
+	return &common.ARecord{
+		Domain:   *record.Name,
+		IP:       *record.Content,
+		TTL:      recordTtl,
+		Prio:     recordPrio,
+		Disabled: recordDisabled,
+	}, nil
+}
+
+func (i *IONOS) getZoneIdForTld(tld string) (string, error) {
+	i.zoneIdMapMutex.Lock()
+	defer i.zoneIdMapMutex.Unlock()
+	if zoneId, ok := i.zoneIdMap[tld]; ok {
+		return zoneId, nil
+	}
+
+	res, err := i.client.GetZonesWithResponse(context.Background())
+	if err != nil {
+		return "", err
+	} else if res.StatusCode() != 200 {
+		return "", errors.New("failed to get zones")
+	}
+
+	if res.JSON200 == nil {
+		return "", errors.New("zones response is empty")
+	}
+
+	zones := *res.JSON200
+
+	for _, zone := range zones {
+		if *zone.Name == tld {
+			if zone.Id == nil || *zone.Id == "" {
+				return "", errors.New("zone id is empty")
+			}
+
+			i.zoneIdMap[tld] = *zone.Id
+			return *zone.Id, nil
+		}
+	}
+
+	return "", errors.New("no zone found")
+}
+
+var recordTypeA = "A"
+
+func assembleTldSubdomainToDomain(tld string, subdomain string) string {
+	if subdomain == "@" || subdomain == "" {
+		return tld
+	}
+	return subdomain + "." + tld
+}
+
+func (i *IONOS) getRecordIdForDomain(zoneId string, domain string) (string, error) {
+	res, err := i.client.GetZoneWithResponse(context.Background(), zoneId, &ionosDnsClient.GetZoneParams{
+		RecordName: &domain,
+		RecordType: &recordTypeA,
+	})
+	if err != nil || res.StatusCode() != 200 {
+		return "", errors.New("failed to get zone")
+	}
+
+	if res.JSON200 == nil {
+		return "", errors.New("zone response is empty")
+	}
+
+	zone := *res.JSON200
+
+	for _, record := range *zone.Records {
+		if *record.Name == domain && *record.Type == ionosDnsClient.RecordTypes("A") {
+			return *record.Id, nil
+		}
+	}
+
+	return "", errors.New("no record found")
+}
+
+func (i *IONOS) GetRecord(tld string, subdomain string) (*common.ARecord, error) {
+	zoneId, err := i.getZoneIdForTld(tld)
+	if err != nil {
+		return nil, err
+	}
+
+	domain := assembleTldSubdomainToDomain(tld, subdomain)
+	recordId, err := i.getRecordIdForDomain(zoneId, domain)
+	if err != nil {
+		return nil, err
+	}
+
+	res, err := i.client.GetRecordWithResponse(context.Background(), zoneId, recordId)
+	if err != nil || res.StatusCode() != 200 || res.JSON200 == nil {
+		return nil, errors.New("failed to get record")
+	}
+
+	record := *res.JSON200
+
+	if record.Name == nil || *record.Name == "" || *record.Name != domain {
+		return nil, errors.New("record name does not match or is empty")
+	}
+
+	if record.Content == nil || *record.Content == "" {
+		return nil, errors.New("record content is empty")
+	}
+
+	needsUpdate := false
+
+	ttl := i.defaultTtl
+	if record.Ttl != nil {
+		ttl = *record.Ttl
+	} else {
+		needsUpdate = true
+	}
+
+	prio := i.defaultPrio
+	if record.Prio != nil {
+		prio = *record.Prio
+	} else {
+		needsUpdate = true
+	}
+
+	disabled := false
+	if record.Disabled != nil {
+		disabled = *record.Disabled
+	} else {
+		needsUpdate = true
+	}
+
+	// realDynDns requires every field to be set, so we need to update the record if any of the fields are missing
+	if needsUpdate {
+		ip := net.ParseIP(*record.Content)
+		if ip == nil {
+			return nil, errors.New("invalid IP address in record content")
+		}
+
+		updatedRecord, err := i.UpdateRecord(tld, subdomain, ip, ttl, prio, disabled)
+		if err != nil {
+			return nil, err
+		}
+		return updatedRecord, nil
+	}
+
+	return &common.ARecord{
+		Domain:   *record.Name,
+		IP:       *record.Content,
+		TTL:      ttl,
+		Prio:     prio,
+		Disabled: disabled,
+	}, nil
 }

pkg/externalIpProvider/plain/plain.go

@@ -2,6 +2,7 @@ package externalIpProvider
 
 import (
 	"errors"
+	"io"
 	"net"
 	"net/http"
 	"net/url"
@@ -40,12 +41,15 @@ func (p *ExternalIpProviderImplPlain) GetExternalIp() (net.IP, error) {
 	}
 
 	if res.StatusCode != 200 {
+		res.Body.Close()
 		return nil, errors.New("unexpected status code")
 	}
 
-	responseBody := make([]byte, res.ContentLength)
+	responseBody, err := io.ReadAll(res.Body)
-	res.Body.Read(responseBody)
+	res.Body.Close()
-	defer res.Body.Close()
+	if err != nil {
+		return nil, err
+	}
 
 	parsedIp := net.ParseIP(string(responseBody))
 	if parsedIp == nil {

renovate.json

@@ -0,0 +1,22 @@
+{
+  "$schema": "https://docs.renovatebot.com/renovate-schema.json",
+  "extends": [
+    "local>t.behrendt/renovate-configs:common",
+    "local>t.behrendt/renovate-configs:action"
+  ],
+  "packageRules": [
+    {
+      "matchPackageNames": [
+        "golang",
+        "gomod",
+        "go"
+      ],
+      "groupName": "go version",
+      "matchUpdateTypes": [
+        "major",
+        "minor",
+        "patch"
+      ]
+    }
+  ]
+}