backend: Fix git_bare bundle download URL sent in email notification (!165) · Merge requests · Platform / Development / swh-vault · GitLab

Snippets Groups Projects

Closed Antoine Lambert requested to merge generated-differential-D8818-source into generated-differential-D8818-target 2 years ago

1 unresolved thread

Previous URL was starting with /api/1/vault/git_bare/ leading to a 404 while the correct one must start with /api/1/vault/git-bare/.

Migrated from D8818 (view on Phabricator)

Activity

Software Heritage Continuous Integration for public repositories @swh-public-ci · 2 years ago

Build is green

Patch application report for D8818 (id=31794)

Rebasing onto d6de3a4d...

Current branch diff-target is up to date.

Changes applied before test

commit b77c5ecef778427829123a499bc3436511b40bfc
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Mon Nov 7 13:50:09 2022 +0100

    backend: Fix git_bare bundle download URL sent in email notification
    
    Previous URL was starting with /api/1/vault/git_bare/ leading to a 404
    while the correct one must start with /api/1/vault/git-bare/.

See https://jenkins.softwareheritage.org/job/DVAU/job/tests-on-diff/256/ for more details.

vlorentz @vlorentz started a thread on the diff 2 years ago

swh/vault/backend.py

         #   the table
         # * use this url for the notification e-mail
         url = "https://archive.softwareheritage.org/api/1/vault/{}/{}/" "raw".format(
             bundle_type, swhid
             bundle_type.replace("_", "-"), swhid

vlorentz @vlorentz · 2 years ago

Maintainer

Merge request was accepted

vlorentz approved this merge request 2 years ago

approved this merge request

Antoine Lambert @anlambert · 2 years ago

Author Maintainer

Merge request was merged

Antoine Lambert closed 2 years ago

closed

Please register or sign in to reply