Skip to content

Fix DockerImageName parsing when tag and digest are used - #11368

Open
bedla wants to merge 1 commit into
testcontainers:mainfrom
bedla:fix-imagename-parsing
Open

bedla wants to merge 1 commit into
testcontainers:mainfrom
bedla:fix-imagename-parsing

Conversation

@bedla

@bedla bedla commented Dec 22, 2025

Copy link
Copy Markdown
Contributor

Hi, based on discussion here #4762 I have prepared this PR.
It seems that combination of tag & digest parsing is broken.
What do you think?
Thx
Ivos

@bedla
bedla requested a review from a team as a code owner December 22, 2025 19:03
@bedla bedla changed the title 4762 Fix DockerImageName parsing when tag and digest are used Fix DockerImageName parsing when tag and digest are used Dec 22, 2025
Comment on lines +68 to +94
// Arguments.of("", "", "myname", ":", null),
// Arguments.of("", "", "myname", ":", "latest"),
// Arguments.of("", "", "repo/myname", ":", null),
// Arguments.of("", "", "repo/myname", ":", "latest"),
// Arguments.of("registry.foo.com:1234", "/", "my-name", ":", null),
// Arguments.of("registry.foo.com:1234", "/", "my-name", ":", "1.0"),
// Arguments.of("registry.foo.com", "/", "my-name", ":", "1.0"),
// Arguments.of("registry.foo.com:1234", "/", "repo_here/my-name", ":", null),
// Arguments.of("registry.foo.com:1234", "/", "repo_here/my-name", ":", "1.0"),
// Arguments.of("1.2.3.4:1234", "/", "repo_here/my-name", ":", null),
// Arguments.of("1.2.3.4:1234", "/", "repo_here/my-name", ":", "1.0"),
// Arguments.of("1.2.3.4:1234", "/", "my-name", ":", null),
// Arguments.of("1.2.3.4:1234", "/", "my-name", ":", "1.0"),
// Arguments.of("", "", "myname", "@", "sha256:1234abcd1234abcd1234abcd1234abcd"),
// Arguments.of("", "", "repo/myname", "@", "sha256:1234abcd1234abcd1234abcd1234abcd"),
// Arguments.of(
// "registry.foo.com:1234",
// "/",
// "repo-here/my-name",
// "@",
// "sha256:1234abcd1234abcd1234abcd1234abcd"
// ),
// Arguments.of("registry.foo.com:1234", "/", "my-name", "@", "sha256:1234abcd1234abcd1234abcd1234abcd"),
// Arguments.of("1.2.3.4", "/", "my-name", "@", "sha256:1234abcd1234abcd1234abcd1234abcd"),
// Arguments.of("1.2.3.4:1234", "/", "my-name", "@", "sha256:1234abcd1234abcd1234abcd1234abcd"),
// Arguments.of("1.2.3.4", "/", "my-name", "@", "sha256:1234abcd1234abcd1234abcd1234abcd"),
// Arguments.of("1.2.3.4:1234", "/", "my-name", "@", "sha256:1234abcd1234abcd1234abcd1234abcd"),

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

???

@simonhir

simonhir commented Oct 2, 2026

Copy link
Copy Markdown

@bedla did the review comment maybe slip through or is there any update on this?
Would be great if this feature would be merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants