From 10abcd8059aad3dc89385ae7008ed1e1c6fb2468 Mon Sep 17 00:00:00 2001 From: Alex Heneveld Date: Sat, 2 Nov 2013 13:33:25 -0700 Subject: [PATCH] JCLOUDS-362 inferring project-id from identity when there is a hyphen --- .../config/GoogleComputeEngineHttpApiModule.java | 18 ++++++++++++++---- 1 file changed, 14 insertions(+), 4 deletions(-) diff --git a/google-compute-engine/src/main/java/org/jclouds/googlecomputeengine/config/GoogleComputeEngineHttpApiModule.java b/google-compute-engine/src/main/java/org/jclouds/googlecomputeengine/config/GoogleComputeEngineHttpApiModule.java index 31f3786..bbf10f0 100644 --- a/google-compute-engine/src/main/java/org/jclouds/googlecomputeengine/config/GoogleComputeEngineHttpApiModule.java +++ b/google-compute-engine/src/main/java/org/jclouds/googlecomputeengine/config/GoogleComputeEngineHttpApiModule.java @@ -101,10 +101,20 @@ protected void bindErrorHandlers() { return MemoizedRetryOnTimeOutButNotOnAuthorizationExceptionSupplier.create(authException, compose(new Function() { public String apply(Credentials in) { - checkState(in.identity.indexOf("@") != 1, - "identity should be in project_id@developer.gserviceaccount.com format"); - - Project project = api.getProjectApi().get(Iterables.get(Splitter.on("@").split(in.identity), 0)); + // ID should be of the form project_id@developer.gserviceaccount.com + // OR (increasingly often) project_id-extended_uid@developer.gserviceaccount.com + // where project_id is the NUMBER; + // HERE we also accept simply "project" as the identity, if no "@" is present; + // this is used in tests, but not sure if it is valid in the wild. + String projectName = in.identity; + if (projectName.indexOf("@") != -1) { + projectName = Iterables.get(Splitter.on("@").split(projectName), 0); + if (projectName.indexOf("-") != -1) { + // if ID is of the form project_id-extended_uid@developer.gserviceaccount.com + projectName = Iterables.get(Splitter.on("-").split(projectName), 0); + } + } + Project project = api.getProjectApi().get(projectName); return project.getName(); } }, creds), seconds, TimeUnit.SECONDS); -- 1.8.4