Skip to content
This repository has been archived by the owner on Mar 16, 2024. It is now read-only.

Fixing partial url for quay.io #38

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

stevenbower
Copy link

Quay.io returns "next page" links like '/v2/....' vs including fully qualified URLs. This patch does the URL join with the registry URL to get a proper fully qualified URL

@stevenbower
Copy link
Author

Fixes #27

@ShashankYadav
Copy link

Any updates on this? It would be great if this fix can get merged into master @amerine @dmathieu

@sbres
Copy link

sbres commented Feb 4, 2020

Sorry to bother you @Myrcurial , but would it be possible to merge it?

@Myrcurial
Copy link

I'm not comfortable with that operation @sbres -- there may be more appropriate people to do this work.

@sbres
Copy link

sbres commented Feb 4, 2020

Sorry, Edward Muller account said to ping you. If there is someone you know could merge this, could you ping him ?

@Myrcurial
Copy link

@sbres Ok - @cfunkhouser can you look at this request from @sbres and see if it's reasonable to do the merge?

@alexwang19
Copy link

Hi, can this change be merged as this would resolve the issue I'm also experiencing with Quay.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants