Skip to content

Conversation

Copy link

Copilot AI commented Sep 26, 2025

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.

Original prompt

For the resolveRegistryHost function in registry.go, consider that the provided host might contain http:// or https:// prefixes.

They should be stripped, either within the function or at the corresponding callsite (consider both options and make an informed decision what is better).

Ensure we have a test case, that eventually the key https://index.docker.io/v1 from an auth config resolves correctly.


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

@kiview kiview closed this Sep 26, 2025
@kiview kiview deleted the copilot/fix-4fb2be3c-8990-497c-b00a-f65c97713972 branch September 26, 2025 11:01
Copilot AI requested a review from kiview September 26, 2025 11:01
Copilot finished work on behalf of kiview September 26, 2025 11:01
@kiview
Copy link
Member

kiview commented Sep 26, 2025

Sorry for that one, this agent task was supposed to go into an entirely different repo 😿 (https://github.com/docker/go-sdk)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants