mirror: fix rename

api/mirror.go

@@ -597,6 +597,7 @@ func apiMirrorsUpdate(c *gin.Context) {
 			if err == nil {
 				return &task.ProcessReturnValue{Code: http.StatusConflict, Value: nil}, fmt.Errorf("unable to rename: mirror %s already exists", b.Name)
 			}
+			remote.Name = b.Name
 		}
 
 		downloader := context.NewDownloader(out)

go.mod

@@ -1,6 +1,6 @@
 module github.com/aptly-dev/aptly
 
-go 1.26
+go 1.25.0
 
 require (
 	github.com/AlekSi/pointer v1.1.0

system/Dockerfile

@@ -2,7 +2,7 @@ FROM debian:trixie-slim
 
 RUN echo 'deb http://deb.debian.org/debian trixie-backports main' > /etc/apt/sources.list.d/backports.list && \
     apt-get update -y && apt-get install -y --no-install-recommends curl gnupg bzip2 xz-utils ca-certificates vim procps \
-        golang-1.26 golang-1.26-go golang-1.26-src \
+        golang-1.25 golang-1.25-go golang-1.25-src \
         make git python3 python3-requests-unixsocket python3-termcolor python3-swiftclient python3-boto3 python3-azure-storage \
         g++ python3-etcd3 python3-plyvel graphviz devscripts sudo dh-golang binutils-i686-linux-gnu binutils-aarch64-linux-gnu \
         binutils-arm-linux-gnueabihf bash-completion zip ruby-dev lintian npm \
@@ -10,8 +10,8 @@ RUN echo 'deb http://deb.debian.org/debian trixie-backports main' > /etc/apt/sou
         gcc-i686-linux-gnu gcc-arm-linux-gnueabihf gcc-aarch64-linux-gnu \
         faketime dput-ng && \
     apt-get clean && rm -rf /var/lib/apt/lists/* && \
-    ln -sf /usr/lib/go-1.26/bin/go /usr/local/bin/go && \
+    ln -sf /usr/lib/go-1.25/bin/go /usr/local/bin/go && \
-    ln -sf /usr/lib/go-1.26/bin/gofmt /usr/local/bin/gofmt
+    ln -sf /usr/lib/go-1.25/bin/gofmt /usr/local/bin/gofmt
 
 RUN useradd -m --shell /bin/bash --home-dir /var/lib/aptly aptly
 RUN sed -i 's/#force_color_prompt=yes/force_color_prompt=yes/' /var/lib/aptly/.bashrc

system/t12_api/mirrors.py

@@ -279,6 +279,78 @@ class MirrorsAPITestEditArchiveURL(APITest):
         self.check_equal(resp.json()['ArchiveRoot'], 'http://repo.aptly.info/system-tests/ftp.ch.debian.org/debian/')
 
 
+class MirrorsAPITestRename(APITest):
+    """
+    PUT /api/mirrors/:name - Rename mirror via update endpoint
+    """
+    def check(self):
+        mirror_name = self.random_name()
+        mirror_desc = {'Name': mirror_name,
+                       'ArchiveURL': 'http://repo.aptly.info/system-tests/packagecloud.io/varnishcache/varnish30/debian/',
+                       'IgnoreSignatures': True,
+                       'Distribution': 'wheezy',
+                       'Components': ['main'],
+                       'Architectures': ['amd64']}
+
+        resp = self.post("/api/mirrors", json=mirror_desc)
+        self.check_equal(resp.status_code, 201)
+
+        # Rename the mirror via PUT
+        new_name = self.random_name()
+        resp = self.put_task("/api/mirrors/" + mirror_name, json={'Name': new_name, 'IgnoreSignatures': True})
+        self.check_task(resp)
+
+        # Old name should no longer exist
+        resp = self.get("/api/mirrors/" + mirror_name)
+        self.check_equal(resp.status_code, 404)
+
+        # New name should exist with correct data
+        resp = self.get("/api/mirrors/" + new_name)
+        self.check_equal(resp.status_code, 200)
+        self.check_subset({'Name': new_name,
+                           'ArchiveRoot': 'http://repo.aptly.info/system-tests/packagecloud.io/varnishcache/varnish30/debian/',
+                           'Distribution': 'wheezy'}, resp.json())
+
+
+class MirrorsAPITestRenameConflict(APITest):
+    """
+    PUT /api/mirrors/:name - Rename mirror to an already existing name should fail with 409
+    """
+    def check(self):
+        mirror_name_a = self.random_name()
+        mirror_name_b = self.random_name()
+
+        mirror_desc_a = {'Name': mirror_name_a,
+                         'ArchiveURL': 'http://repo.aptly.info/system-tests/packagecloud.io/varnishcache/varnish30/debian/',
+                         'IgnoreSignatures': True,
+                         'Distribution': 'wheezy',
+                         'Components': ['main'],
+                         'Architectures': ['amd64']}
+
+        mirror_desc_b = {'Name': mirror_name_b,
+                         'ArchiveURL': 'http://repo.aptly.info/system-tests/packagecloud.io/varnishcache/varnish30/debian/',
+                         'IgnoreSignatures': True,
+                         'Distribution': 'wheezy',
+                         'Components': ['main'],
+                         'Architectures': ['amd64']}
+
+        resp = self.post("/api/mirrors", json=mirror_desc_a)
+        self.check_equal(resp.status_code, 201)
+
+        resp = self.post("/api/mirrors", json=mirror_desc_b)
+        self.check_equal(resp.status_code, 201)
+
+        # Try to rename mirror A to mirror B's name — should fail with 409
+        resp = self.put("/api/mirrors/" + mirror_name_a, json={'Name': mirror_name_b, 'IgnoreSignatures': True})
+        self.check_equal(resp.status_code, 409)
+        self.check_in('unable to rename', resp.json()['error'])
+
+        # Mirror A should still exist under its original name
+        resp = self.get("/api/mirrors/" + mirror_name_a)
+        self.check_equal(resp.status_code, 200)
+        self.check_subset({'Name': mirror_name_a}, resp.json())
+
+
 class MirrorsAPITestEditFlatMirrorUdebs(APITest):
     """
     POST /api/mirrors/{name} - Edit flat mirror with udebs (should fail)