concourse / concourse/docker-image-resource

Doesn't cope with private registry with no '.' in hostname

Open
#90 8 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

stack/docker
Dominant language
Go
Stars
162
Forks
250
PR merge metrics
No merged PRs in 30d

Description

If your private registry does not contain a '.', then due to the logic in the following code, the function assumes that you are not using a private registry and attempts to use docker hub.

https://github.com/concourse/docker-image-resource/blob/e6ffb336064e606b1d802a4ddc9cdab69553d0bd/assets/common.sh#L105

Scenarios like this can crop up when concourse and your registry are on the same network, say within a docker network.

I'm using nexus for my private docker registry/proxy, but if I use the docker FQDN when logging in I get an HTTP 400; yes this is a Nexus issue, but the assumption in the code above is still wrong.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start in assets/common.sh at line 105 and inspect how registry hostnames are classified before Docker Hub is selected. Reproduce the case with a private registry hostname that has no dot; done means the resource selects that registry rather than Docker Hub.

Written by the indexing model from the issue text.

Assessment

Tech stack
docker, shell
Domain
devops
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.