Allow `gspread.oauth()` to look for file in another directory

Is your feature request related to a problem? Please describe. Not really a problem, but I got bitten a bit by it. I used to be using a service account and service_account allows users to store google credentials where they want. This is particularly relevant for the flow I have implemented in my CLI tool to import google sheets to SQL databases (sheetwork) where I require users to put credentials into a .sheetwork folder.

gspread.oauth doesn’t take arguments for filename, but since it uses load_credentials() https://github.com/burnash/gspread/blob/master/gspread/auth.py#L93 I would assume this is possible.

Is there a reason why this might not be allowed? Could it break a few things?

Describe the solution you’d like allow oaut() to take a filename argument that is passed to load_credentials

If you agree that this would be possible, I’d be happy to contribute to the change.

[akochkarev]

Is your feature request related to a problem? Please describe. Not really a problem, but I got bitten a bit by it. I used to be using a service account and service_account allows users to store google credentials where they want. This is particularly relevant for the flow I have implemented in my CLI tool to import google sheets to SQL databases (sheetwork) where I require users to put credentials into a .sheetwork folder.

gspread.oauth doesn’t take arguments for filename, but since it uses load_credentials() https://github.com/burnash/gspread/blob/master/gspread/auth.py#L93 I would assume this is possible.

Is there a reason why this might not be allowed? Could it break a few things?

Describe the solution you’d like allow oaut() to take a filename argument that is passed to load_credentials

If you agree that this would be possible, I’d be happy to contribute to the change.

I faced the same problem when start to use gspread. Workaround is to change default creds location by modifying gspread settings and default function parameters. Not very elegant, but working solution:

import gspread.auth as ga
import mysettings as cfg

def changeUserCredsStorage():

    ga.DEFAULT_CONFIG_DIR = cfg.DEFAULT_SECRETS_DIR

    ga.DEFAULT_CREDENTIALS_FILENAME = os.path.join(
        ga.DEFAULT_CONFIG_DIR, 'credentials.json'
    )
    ga.DEFAULT_AUTHORIZED_USER_FILENAME = os.path.join(
        ga.DEFAULT_CONFIG_DIR, 'authorized_user.json'
    )
    ga.DEFAULT_SERVICE_ACCOUNT_FILENAME = os.path.join(
        ga.DEFAULT_CONFIG_DIR, 'service_account.json'
    )

    ga.load_credentials.__defaults__ = (ga.DEFAULT_AUTHORIZED_USER_FILENAME,)
    ga.store_credentials.__defaults__ = (ga.DEFAULT_AUTHORIZED_USER_FILENAME, 'token')

But now i switch to service accounts and do not use this.

[bastienboutonnet]

I believe when @burnash gets around to giving another look at the PR it should be good as all the changes they asked for have been implemented.