http://codereview.appspot.com/6441118/diff/1/oauth2client/gce.py File oauth2client/gce.py (right): http://codereview.appspot.com/6441118/diff/1/oauth2client/gce.py#newcode34 oauth2client/gce.py:34: META = 'http://metadata/0.1/meta-data/service-accounts/default/acquire{?scope}' 'default' could be a parameter. http://codereview.appspot.com/6441118/diff/1/oauth2client/gce.py#newcode37 ...
http://codereview.appspot.com/6441118/diff/1/oauth2client/gce.py File oauth2client/gce.py (right): http://codereview.appspot.com/6441118/diff/1/oauth2client/gce.py#newcode34 oauth2client/gce.py:34: META = 'http://metadata/0.1/meta-data/service-accounts/default/acquire{?scope}' It could be, but we can ...
http://codereview.appspot.com/6441118/diff/1/oauth2client/gce.py
File oauth2client/gce.py (right):
http://codereview.appspot.com/6441118/diff/1/oauth2client/gce.py#newcode34
oauth2client/gce.py:34: META =
'http://metadata/0.1/meta-data/service-accounts/default/acquire{?scope}'
It could be, but we can always add that in the future. The reason I don't want
to add it now is because I don't know what non-default values would look like,
would they be project ids, or email addresses, or maybe private key finger
prints...
On 2012/08/09 13:54:47, aiuto wrote:
> 'default' could be a parameter.
http://codereview.appspot.com/6441118/diff/1/oauth2client/gce.py#newcode37
oauth2client/gce.py:37: class AppAssertionCredentials(AssertionCredentials):
Actually this already sits in the 'gce' module, how about simply
AssertionCredentials?
On 2012/08/09 13:54:47, aiuto wrote:
> I think we might be better off with ComputeEngineAssertionCredentials.
http://codereview.appspot.com/6441118/diff/1/oauth2client/gce.py#newcode40
oauth2client/gce.py:40: This object will allow an Compute Engine instance to
identify itself to
On 2012/08/09 13:54:47, aiuto wrote:
> allow a Compute ...
Done.
http://codereview.appspot.com/6441118/diff/1/oauth2client/gce.py File oauth2client/gce.py (right): http://codereview.appspot.com/6441118/diff/1/oauth2client/gce.py#newcode84 oauth2client/gce.py:84: response, content = http_request(uri) Can this refresh use the ...
http://codereview.appspot.com/6441118/diff/1/oauth2client/gce.py File oauth2client/gce.py (right): http://codereview.appspot.com/6441118/diff/1/oauth2client/gce.py#newcode84 oauth2client/gce.py:84: response, content = http_request(uri) Right, is there no common ...
http://codereview.appspot.com/6441118/diff/1/oauth2client/gce.py File oauth2client/gce.py (right): http://codereview.appspot.com/6441118/diff/1/oauth2client/gce.py#newcode84 oauth2client/gce.py:84: response, content = http_request(uri) _refresh() is defined in the ...
http://codereview.appspot.com/6441118/diff/1/oauth2client/gce.py
File oauth2client/gce.py (right):
http://codereview.appspot.com/6441118/diff/1/oauth2client/gce.py#newcode37
oauth2client/gce.py:37: class AppAssertionCredentials(AssertionCredentials):
On 2012/08/09 14:37:40, jcgregorio_google wrote:
> Actually this already sits in the 'gce' module, how about simply
> AssertionCredentials?
Won't you have a type name clash?
class AssertionCredentials(AssertionCredentials)
unless you change the import of AssertionCredentials?
AppAssertionCredentials is fine.
http://codereview.appspot.com/6441118/diff/1/oauth2client/gce.py File oauth2client/gce.py (right): http://codereview.appspot.com/6441118/diff/1/oauth2client/gce.py#newcode84 oauth2client/gce.py:84: response, content = http_request(uri) Oh, I see it is ...
On 2012/08/09 18:13:48, aiuto wrote: > lgtm Committed at http://code.google.com/p/google-api-python-client/source/detail?r=51f8b6a497ba61577ccb2239ac78b8452e759460
Issue 6441118: Add support for GCE service accounts
(Closed)
Created 13 years ago by jcgregorio_google
Modified 13 years ago
Reviewers: aiuto, jcgregorio_google, Ali Afshar
Base URL:
Comments: 12