> @@ -101,10 +101,20 @@ protected void bindErrorHandlers() {
> return
> MemoizedRetryOnTimeOutButNotOnAuthorizationExceptionSupplier.create(authException,
> compose(new Function<Credentials, String>() {
> public String apply(Credentials in) {
> - checkState(in.identity.indexOf("@") != 1,
> - "identity should be in
> project...@developer.gserviceaccount.com format");
> -
> - Project project =
> api.getProjectApi().get(Iterables.get(Splitter.on("@").split(in.identity),
> 0));
> + // ID should be of the form
> project...@developer.gserviceaccount.com
> + // OR (increasingly often)
> project_id-extended_...@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.
> Yeah that was likely the intention
I would suspect that the tests were simply not set up correctly and it wasn't
fixed because the tests happened to pass. But if indeed we shouldn't actually
be trying to enforce this restriction, then #12 should be closed.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-google/pull/9/files#r7448227
