Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Define path of bigquery_credentials.dat by environment var #86

Closed
jonathansp opened this issue Sep 12, 2017 · 8 comments
Closed

Define path of bigquery_credentials.dat by environment var #86

jonathansp opened this issue Sep 12, 2017 · 8 comments

Comments

@jonathansp
Copy link
Contributor

In a shared environment, the use of hard-coded bigquery_credentials.dat file name could be a problem. We should be able to define this path by using an environment variable, for instance, PANDAS_GBQ_CREDENTIALS_FILE.

@max-sixty
Copy link
Contributor

Can we default to GOOGLE_APPLICATION_CREDENTIALS, or does this need to be additional? https://developers.google.com/identity/protocols/application-default-credentials

@parthea
Copy link
Contributor

parthea commented Sep 12, 2017

Thanks for reporting this. I think this is a duplicate of #41. We have an improvement ready (#83) and I'm hoping to merge it tonight if there are no objections. Could you add comments to #83?

@andrelu
Copy link

andrelu commented Sep 12, 2017

IMO it would be interesting to have a configurable path. In #83 we cannot set a default directory to store our config, you ll always store in ~/.config/pandas_gbq... this is not interesting when having datascience isolate notebooks environments. Beside that, you always have to pass the credentials when executing read_gbq.

@andrelu
Copy link

andrelu commented Sep 12, 2017

@MaximilianR I think that GOOGLE_APPLICATION_CREDENTIALS env is used for a different kind of credential.

@jonathansp
Copy link
Contributor Author

jonathansp commented Sep 12, 2017

Thank you guys for replying.

@parthea @MaximilianR could you please consider both approaches? Maybe it'll be tough asking our users for sending credentials_path parameter every time they need to exec a query.

@MaximilianR, as @andrelu pointed out, GOOGLE_APPLICATION_CREDENTIALS seems to have another meaning.

@parthea
Copy link
Contributor

parthea commented Sep 12, 2017

@jonathansp You're right. Users may not like sending credentials_path parameter every time they need to exec a query. I'll modify #83 tonight to use an environment variable, as you've done in #86, and remove credentials_path altogether. I'll start by building on top of your changes in #86 so we can keep that PR open to maintain authorship.

@max-sixty
Copy link
Contributor

I think that GOOGLE_APPLICATION_CREDENTIALS env is used for a different kind of credential.

Yes, you're right

@parthea
Copy link
Contributor

parthea commented Sep 19, 2017

Closed via dbfb4e9

@parthea parthea closed this as completed Sep 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants