Skip to content

Refactor to remove old auth.console_flow references#1493

Open
mafrosis wants to merge 1 commit into
burnash:masterfrom
mafrosis:oauth-cleanup
Open

Refactor to remove old auth.console_flow references#1493
mafrosis wants to merge 1 commit into
burnash:masterfrom
mafrosis:oauth-cleanup

Conversation

@mafrosis

Copy link
Copy Markdown

No description provided.

@alifeee

alifeee commented Jul 23, 2024

Copy link
Copy Markdown
Collaborator

thanks for the suggestion!

What motivates this change? You remove a keyword argument to some functions. Some people could rely on using these, no?

@alifeee alifeee requested a review from lavigne958 July 23, 2024 14:52
@lavigne958

Copy link
Copy Markdown
Collaborator

thanks for the suggestion!

What motivates this change? You remove a keyword argument to some functions. Some people could rely on using these, no?

no people can't rely on them as the flow is not working anymore with Google Authentication system.

Thank you for this clean, I forgot about it.

so far no one should be able to use this flow for months, though we should probably keep it for a major release just in case ?

@mafrosis

Copy link
Copy Markdown
Author

Hey, sorry for the lack of description 😄

For future readers:
googleapis/google-auth-library-python-oauthlib@1fb16be

I was working on some cross-library compatibility regarding Google API auth flows and tokens and such (gspread & kuzmoyev/google-calendar-simple-api & jeremyephron/simplegmail), and I noticed all this effectively dead code in the gspread auth library. Just doing the good citizen open source thing and sending out a PR for y'all

Thanks for gspread, it's awesome 🫡

@lavigne958

Copy link
Copy Markdown
Collaborator

thanks for this clean-up, effectively it's dead code, though I'd rather merge it in the next major release just in case, we've seen many issues like this before where it should be ok to merge this and it was not for a few users.

We'll keep this PR open for now until we make next major release and then merge it.

thank in advance for this contribution 🙃

@Chris113113

Copy link
Copy Markdown

Ping on this PR, I just spent a morning trying to use the documented console_flow before realizing that it's non-functional.

@mafrosis / @lavigne958

@mafrosis

mafrosis commented Jul 1, 2025

Copy link
Copy Markdown
Author

Looks like this missed the 6.2 release - @lavigne958, I'll rebase this shortly and update the PR. Maybe can include in 6.3?

@mafrosis

mafrosis commented Aug 6, 2025

Copy link
Copy Markdown
Author

@burnash @lavigne958 PR updated, please merge

darkfiberiru pushed a commit to darkfiberiru/gspread that referenced this pull request Nov 25, 2025
…ences

This PR removes deprecated OAuth authentication code that has been non-functional for months due to changes in Google's authentication system. The console_flow authentication method is effectively dead code.

Approved by: lavigne958
Status: Ready for merge (rebased August 6, 2025)
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

Successfully merging this pull request may close these issues.

4 participants