Inedo Community Forums Forums
    • Recent
    • Tags
    • Popular
    • Login

    Welcome to the Inedo Forums! Check out the Forums Guide for help getting started.

    If you are experiencing any issues with the forum software, please visit the Contact Form on our website and let us know!

    Encode URI incorrectly cause GCR connector not working

    Scheduled Pinned Locked Moved Support
    4 Posts 2 Posters 16 Views
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • PMExtraP Offline
      PMExtra
      last edited by PMExtra

      Re: ProGet Docker Connector to gcr.io not working

      I met the same problem. After analysed the captured traffic, I found that ProGet encoded the URI incorrectly.

      The colon (:) does not need to be encoded in URI (ref https://stackoverflow.com/questions/2053132/is-a-colon-safe-for-friendly-url-use).

      GCR needs the original colon rather than %3A, but ProGet encoded it. So we got 404 response from GCR.

      Example:

      This is a good request:
      https://gcr.io/v2/kaniko-project/executor/manifests/sha256:034f15e6fe235490e64a4173d02d0a41f61382450c314fffed9b8ca96dff66b2

      But ProGet encoded the colon:
      https://gcr.io/v2/kaniko-project/executor/manifests/sha256%3A034f15e6fe235490e64a4173d02d0a41f61382450c314fffed9b8ca96dff66b2

      atrippA 1 Reply Last reply Reply Quote 0
      • atrippA Offline
        atripp inedo-engineer @PMExtra
        last edited by atripp

        Hi @PMExtra ,

        Thanks for the details on this, very helpful :)

        We actually recently discovered this too (though a different underlying problem) - it was should be addressed in 2022.27 via PG-2321.

        Can you try that and see if it helps? We can investigate further if not.

        Cheers,
        Alana

        1 Reply Last reply Reply Quote 0
        • PMExtraP Offline
          PMExtra
          last edited by

          The problem still exists in version 23.0.7

          atrippA 1 Reply Last reply Reply Quote 0
          • atrippA Offline
            atripp inedo-engineer @PMExtra
            last edited by

            Hi @PMExtra ,

            Looks like this didn't make it to the 2023 codebase; I've just merged it in via PG-2388 (shipping this friday in ProGet 2023.8).

            Cheers,
            Alana

            1 Reply Last reply Reply Quote 0

            Hello! It looks like you're interested in this conversation, but you don't have an account yet.

            Getting fed up of having to scroll through the same posts each visit? When you register for an account, you'll always come back to exactly where you were before, and choose to be notified of new replies (either via email, or push notification). You'll also be able to save bookmarks and upvote posts to show your appreciation to other community members.

            With your input, this post could be even better 💗

            Register Login
            • 1 / 1
            • First post
              Last post
            Inedo Website Home • Support Home • Code of Conduct • Forums Guide • Documentation