-
Notifications
You must be signed in to change notification settings - Fork 867
main: Add support for overriding HTTP User-Agent #2724
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I… don’t think that identifying use cases via User-Agent generalizes at all. Podman is not going to use a different User-Agent depending on operation, for example.
But, *shrug*, c/image does have DockerRegistryUserAgent
so it makes sense for Skopeo to expose an option.
I read https://www.rfc-editor.org/rfc/rfc7231.html#section-5.5.3 to mean that (
…)
is a comment syntax, that is not what I’d expect here.
Yes, it would be unusual. But conceptually think of this as for rich enough use cases where one has a proper "application" on top of skopeo - it might be e.g. a registry which supports auto-mirroring from another registry and forks skopeo as a binary instead of vendoring the libraries because it doesn't want to hop on the CVE train.
Fixed, thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
LGTM, but I’d like a second pair of eyes.
- Log data via structured logging - Include user agent (to test out containers/skopeo#2724 ) Signed-off-by: Colin Walters <[email protected]>
9dfc64b
to
169091e
Compare
Added an |
Who would that be? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Still hanging watching : ^)
/lgtm |
Ah, this doesn't work anymore as I am not maintainer anymore |
AFAICS you do still have an admin role in GitHub, but we never had the bot set up. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM after the lint passes. Thanks!
I want this for bootc-dev/bootc#1686 so we can distinguish pulls there. But more generally it's can be a good idea for people writing scripts using skopeo to set custom user agents so that registries can more easily trace which actors are performing tasks. Assisted-by: Claude Code Signed-off-by: Colin Walters <[email protected]>
I want this for bootc-dev/bootc#1686 so we can distinguish pulls there.
But more generally it's can be a good idea for people writing scripts using skopeo to set custom user agents so that registries can more easily trace which actors are performing tasks.