Skip to content

Fix dependency proxy image prefix

Steve Abrams requested to merge 341888-fix-dep-prox-prefix into master

What does this MR do and why?

Describe in detail what your merge request does and why.

In !69114 (diffs) a bug was introduced that changed the Dependency Proxy image prefix that is displayed in the UI/API to include /groups, which is not correct and will not work with the Dependency Proxy. This fixes the prefix to not include /groups.

Screenshots or screen recordings

Before After
Screen_Shot_2021-09-30_at_2.33.23_PM Screen_Shot_2021-09-30_at_2.31.49_PM

How to set up and validate locally

In the rails console ensure /groups/ is not in the prefix returned:

[6] pry(main)> Group.first.dependency_proxy_image_prefix
=> "gdk.test:3001/gitlab-org/dependency_proxy/containers"

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #341888 (closed)

Merge request reports