Skip to content

registry: deprecate APIEndpoint.Version and APIVersion type#46359

Merged
thaJeztah merged 1 commit intomoby:masterfrom
thaJeztah:deprecate_endpoint_version
Aug 29, 2023
Merged

registry: deprecate APIEndpoint.Version and APIVersion type#46359
thaJeztah merged 1 commit intomoby:masterfrom
thaJeztah:deprecate_endpoint_version

Conversation

@thaJeztah
Copy link
Member

This field was used when the code supported both "v1" and "v2" registries. We no longer support v1 registries, and the only v1 endpoint that's still used is for the legacy "search" endpoint, which does not use the APIEndpoint type.

As no code is using this field, and the value will always be set to "v2", we can deprecated the Version field.

I'm keeping this field for 1 release, to give notice to any potential external consumer, after which we can delete it.

- A picture of a cute animal (not mandatory but encouraged)

This field was used when the code supported both "v1" and "v2" registries.
We no longer support v1 registries, and the only v1 endpoint that's still
used is for the legacy "search" endpoint, which does not use the APIEndpoint
type.

As no code is using this field, and the value will always be set to "v2",
we can deprecated the Version field.

I'm keeping this field for 1 release, to give notice to any potential
external consumer, after which we can delete it.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah force-pushed the deprecate_endpoint_version branch from d11154f to d43e617 Compare August 29, 2023 10:49
@thaJeztah thaJeztah merged commit acdad37 into moby:master Aug 29, 2023
@thaJeztah thaJeztah deleted the deprecate_endpoint_version branch August 29, 2023 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants