Skip to content

Commit

Permalink
Merge pull request #144 from javabrett/auto-method-remove-cloud
Browse files Browse the repository at this point in the history
Exclude explicit method "cloud" search for compute_engine.Credentials().
  • Loading branch information
martindurant committed Apr 9, 2019
2 parents 0847583 + 6b94644 commit 523eb65
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
3 changes: 3 additions & 0 deletions docs/source/index.rst
Original file line number Diff line number Diff line change
Expand Up @@ -98,6 +98,9 @@ Several modes of authentication are supported:
- you can also generate tokens via Oauth2 in the browser using ``token='browser'``,
which gcsfs then caches in a special file, ~/.gcs_tokens, and can subsequently be accessed with ``token='cache'``.

- anonymous only access can be selected using ``token='anon'``, e.g. to access
public resources such as 'anaconda-public-data'.

The acquired session tokens are *not* preserved when serializing the instances, so
it is safe to pass them to worker processes on other machines if using in a
distributed computation context. If credentials are given by a file path, however,
Expand Down
2 changes: 1 addition & 1 deletion gcsfs/core.py
Original file line number Diff line number Diff line change
Expand Up @@ -426,7 +426,7 @@ def connect(self, method=None):
'browser', None]:
self._connect_token(method)
elif method is None:
for meth in ['google_default', 'cache', 'cloud', 'anon']:
for meth in ['google_default', 'cache', 'anon']:
try:
self.connect(method=meth)
if self.check_credentials and meth != 'anon':
Expand Down

0 comments on commit 523eb65

Please sign in to comment.