Add support for bare hostname as endpoint, fix unnecessary ns param inclusion

rancher / wharfie

Pulls and unpacks a container image to the local filesystem. Usable both as a standalone CLI tool, and as a go module.

Apache License 2.0

25 stars 14 forks source link

Closed brandond closed 4 months ago

brandond commented 4 months ago

Add support for endpoints without scheme or as bare hostname. containerd supports this, but wharfie has not historically allowed for this.
Don't set ns param if the request is not being proxied. Some namespace-aware registries have problems handling the request if the ns parameter is set to the same value as the hostname. We should only set the ns parameter when the request is being sent to a non-default endpoint.

internal-ref: SURE-7640