fix: tag regex should allow ports

[artificial-intelligence]

Closes: #3195
Related: opencontainers/distribution-spec#498

notice: there is a false positive warning from ruff during make test:

docker/utils/build.py:14:33: W605 [*] Invalid escape sequence: `\.`

but it's a valid escape sequence, because it's a regex, had no idea yet how to fix this without introducing more complexity.

[artificial-intelligence]

further notice:

I did fix the ruff warning about \/ because that escape sequence is not needed.

[LombardiDaniel]

Make sure to add a simple test like:

def test_build_with_ports_on_tag(self):
        self.client.build(".", tag="registry.example.com:443/my-img:latest")

on tests/build/api_build_and_test.py. And why not use raw string literals? i.e.

r"my-regex"

instead of

"my-regex"

[milas]

Thanks for the bugfix!

I addressed the lint issue (it was valid but very subtle btw: needed to use r"" strings each time for the concatenation 😵) and added some test cases from the distribution/reference codebase.

(There's some xfails on the IPv6 cases which can be addressed separately.)

[SecT0uch]

When can we expect a release including this fix ?

[artificial-intelligence]

Thanks @milas for taking over (I was first on PTO, then this issue was drowned in my inbox due to other issues)!

a new release would be highly appreciated for our downstream usage!