Skip to content

Remove oob fallback and launchy liveness check #11

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

Merged
merged 3 commits into from
May 14, 2025

Conversation

carl-codeorg
Copy link
Contributor

@carl-codeorg carl-codeorg commented May 9, 2025

The out of band Oauth method has been blocked by Google since Jan 1, 2023.

Screenshot 2025-05-08 at 11 50 25 AM

This PR removes the fallback to out of band authentication and the message instructing the user to export an oauth code environment variable, since it no longer works.

Also removes the launchy liveness check, since at least for some users, the liveness check always failed. This check also caused the local oauth server to fail, which ultimately made it impossible to log in. Without the Launchy check, the server will still be shut down after the success request is served.

@cat5inthecradle
Copy link
Contributor

I think this makes sense. The out-of-band approach isn't supported by Google anymore, correct?

@carl-codeorg
Copy link
Contributor Author

carl-codeorg commented May 13, 2025

I think this makes sense. The out-of-band approach isn't supported by Google anymore, correct?

Correct - it's officially EOL'd by Google and results in a 400 error. Just updated the description on this PR, didn't realize I had only posted that info in my other PR.

@carl-codeorg carl-codeorg marked this pull request as ready for review May 14, 2025 15:42
@cat5inthecradle
Copy link
Contributor

Bumped the version, merging then doing a manual release.

@cat5inthecradle cat5inthecradle merged commit b0cb3a9 into main May 14, 2025
2 checks passed
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.

2 participants