@eriwen @bmuschko would appreciate your input on this.

Thanks for providing the pull request. We are going to prioritize soon.

Is there a way you could write an integration that demonstrate that the implementation is now more efficient and performant?

@bmuschko I can add an integration test that makes sure that the GcsClient is not attempting to list all bucket for single resource. Performance impact can only be measured by communicating with GCS directly and I don't think it would be good to do in tests ;]

@bmuschko I have updated GCS Maven integration tests with an additional test that triggers the case where repository is missing the root Maven metadata and Gradle falls back to listing files iteratively. The server stub was updated to verify that only requests with a specific prefix are used when listing module resources.

Bump @bmuschko

@eleventigerssc Thanks for the excellent contribution, I've finished review and everything looks fine. I'm waiting for our CI feedback then I can merge it.

The only question is unrelated to this PR, in your previous contribution: https://github.com/eleventigerssc/gradle/blob/4da6ba7cfd72ab0027c890f9cc9e3db1e26310f0/subprojects/resources-gcs/src/main/java/org/gradle/internal/resource/transport/gcp/gcs/GcsClient.java#L162

Is this path = URLDecoder.decode(uri.getPath(), "UTF-8"); redundant? Looks like URI.getPath()'s result is already decoded:

    /**
     * Returns the decoded path component of this URI.
     *
     * <p> The string returned by this method is equal to that returned by the
     * {@link #getRawPath() getRawPath} method except that all sequences of
     * escaped octets are <a href="#decode">decoded</a>.  </p>
     *
     * @return  The decoded path component of this URI,
     *          or {@code null} if the path is undefined
     */
    public String getPath() {

@blindpirate good point re: double decode. It looks like a plain oversight from my end however we should check why the original GCS support PR had manual / char replacement: https://github.com/gradle/gradle/pull/797/files#diff-e0dee2401703fdb7ba0f18e4ce0900ccR86 - maybe URI decode works differently?

@eleventigerssc Got it. Thanks for your reply.

@eleventigerssc Merged, thanks!

@blindpirate awesome, thank you!