-
Notifications
You must be signed in to change notification settings - Fork 653
This issue was moved to a discussion.
You can continue the conversation there. Go to discussion →
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
Todo: Reverse engineer new Facebook login flow. #244
Comments
New URL client requests in browser is: |
Also, this login flow seems to be exclusive to the desktop app. Mobile app uses a different flow. |
Ok, so here's how the auth flow works:
where the This behaviour shouldn't be particularly hard to implement in librespot. There are no API keys/special headers involved with the endpoint at the time of writing, so librespot could fairly easily adopt a I may get around to implementing this, but as mentioned previously, I don't use it, so it could be a while. |
Works nicely (librespot-org/librespot-java#16)! I'll leave up to you when to close this issue. |
Cheers, we'll leave this open for now. Currently Facebook auth isn't actually implemented in librespot, but it comes of the category of useful to have, hence will leave it here until it gets implemented at some point. |
Heads up: Sasha's analysis in #244 (comment) is no longer accurate, all of these URLs are now redirects to a deprecation notice page. (see hrkfdn/ncspot#310 ) |
If I get some time I'll take a look at the new process. I wish they'd stop fixing stuff that wasn't broken... |
This issue was moved to a discussion.
You can continue the conversation there. Go to discussion →
The Facebook login flow described in the docs is outdated. This is the result of visiting that endpoint:
![](https://user-images.githubusercontent.com/14827249/45039417-e1b84180-b063-11e8-8218-6c588ba377ee.png)
The text was updated successfully, but these errors were encountered: