Skip to content
This repository was archived by the owner on Mar 31, 2022. It is now read-only.
This repository was archived by the owner on Mar 31, 2022. It is now read-only.

ImageRef.parseRegistryUrl:112 #389

@chenkaishui

Description

@chenkaishui

static String parseRegistryUrl(final String url) {
if (url.equals("docker.io") || url.equals("index.docker.io")) {
return DEFAULT_REGISTRY_URL;
} else if (url.matches("(^|(\w+)\.)gcr\.io$") || url.equals("gcr.kubernetes.io")) {
// GCR uses https
return "https://" + url;
} else if (url.equals("quay.io")) {
// Quay doesn't use any protocol in credentials file
return "quay.io";
} else if (!url.contains("http://") && !url.contains("https://")) {
// Assume https
return "https://" + url;
} else {
return url; **//this may not run anytime,because:line 55,56 **
}
}

public ImageRef(final String image){
55: final String[] parts = image.split("/", 2)
56:if (parts.length > 1 && isRegistry(parts[0])) ** It's sure image not contains "http://" or "https://"**
this.registry = parts[0];
this.registryUrl = parseRegistryUrl(parts[0]);
} else {
this.registry = DEFAULT_REGISTRY;
this.registryUrl = DEFAULT_REGISTRY_URL;
}
}

private static boolean isRegistry(String part) {
return part.contains(".");
}

com.spotify dockerfile-maven 1.4.14-SNAPSHOT dockerfile-maven-plugin maven-plugin

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions