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

Add alternate redirect_uri support for broken instances #512

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

rjp
Copy link

@rjp rjp commented Dec 8, 2024

Pixelfed instances do not handle OOB OAuth correctly meaning you can't currently login to them using toot (e.g. pixelfed/pixelfed#2522 )

This is a fudge to workaround that by allowing you to specify an alternate redirect_uri for broken servers which can take the HTTP redirect issued by Pixelfed and let you grab the code.

If you don't have a handy HTTP server, you can use http://localhost and your browser (at least Safari and Chrome) will have the code in the address bar for copying and pasting into toot.

I'm not 100% sure about how an empty redirect_uri is handled in load_app - it feels like supplying -r should set that instead of OOB on an app without an explicit setting but I don't think it's a situation that's going to crop up that often anyway.

(Apologies if this is a Pythonic mess, it's not one of my main languages)

rjp added 3 commits December 8, 2024 13:05
Pixelfed instances do not handle OOB OAuth correctly meaning you can't
currently login to them using `toot` (e.g. pixelfed/pixelfed#2522 )

This is a fudge to workaround that by allowing you to specify an
alternate `redirect_uri` for broken servers which can take the HTTP
redirect issued by Pixelfed and let you grab the code.

If you don't have a handy HTTP server, you can use `http://localhost`
and your browser (at least Safari and Chrome) will have the code in
the address bar for copying and pasting into `toot`.
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.

1 participant