Skip to content

[pull] master from moby:master#489

Merged
pull[bot] merged 5 commits intoadamlaska:masterfrom
moby:master
Mar 14, 2026
Merged

[pull] master from moby:master#489
pull[bot] merged 5 commits intoadamlaska:masterfrom
moby:master

Conversation

@pull
Copy link

@pull pull bot commented Mar 14, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

thaJeztah and others added 5 commits March 14, 2026 00:51
- Prevent panic when calling .Proto() on zero values
- Don't iterate on zero-value port-ranges

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
This brings back the Port() method that was defined on the nat.Port type,
which was used in many places to (e.g.) do a network.JoinHostPort.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Call out that users should check if the port/port-range is valid
before consuming the value returned by String().

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
api/types/network: Port,PortRange: don't panic on zero values, add Port.Port()
@pull pull bot locked and limited conversation to collaborators Mar 14, 2026
@pull pull bot added the ⤵️ pull label Mar 14, 2026
@pull pull bot merged commit 05e2957 into adamlaska:master Mar 14, 2026
9 of 12 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant