finamp icon indicating copy to clipboard operation
finamp copied to clipboard

Quick Connect/SSO Support

Open alx-alexpark opened this issue 3 years ago • 19 comments
trafficstars

I use an sso plugin, making the normal password auth disabled. Is there a way to simply add an auth token or api key to get the app working?

alx-alexpark avatar Nov 07 '22 17:11 alx-alexpark

I could probably add a way to pass an API key manually, although proper quick connect support would be much better

jmshrv avatar Nov 22 '22 16:11 jmshrv

I am actually somewhat familiar with flutter, I might try to implement it myself. On Tue, Nov 22, 2022 at 10:09 James Harvey @.***> wrote:

I could probably add a way to pass an API key manually, although proper quick connect support would be much better

— Reply to this email directly, view it on GitHub https://github.com/jmshrv/finamp/issues/354#issuecomment-1323912434, or unsubscribe https://github.com/notifications/unsubscribe-auth/APGY35GI6XEVZ3WXBGXVFPTWJTV3NANCNFSM6AAAAAARZNHX3U . You are receiving this because you authored the thread.Message ID: @.***>

alx-alexpark avatar Nov 23 '22 00:11 alx-alexpark

@alx-alexpark are you actively working on this? I'm tempted to join as well, or start my own branch if you're not working on it atm.

Flexusma avatar Dec 25 '22 11:12 Flexusma

Hi, author of the SSO plugin here. I'm wondering about the status of this as well :)

Although you can set the fallback login mechanism and allow the user to both have a Jellyfin password as well as long in through SSO, this isn't an ideal solution.

9p4 avatar Mar 26 '23 18:03 9p4

I think I might just start working on this. I don't know if I'll have the time to finish it but we'll see

Flexusma avatar Mar 26 '23 18:03 Flexusma

Has anyone started working on this? I might start a branch if not, it'd be a great feature to have :)

xkcdstickfigure avatar Dec 26 '23 03:12 xkcdstickfigure

Has anyone started working on this? I might start a branch if not, it'd be a great feature to have :)

I didn’t end up working on this. I have not had the resources to work on this :/. If you end up doing this though, lmk I’d be happy to help :)

Flexusma avatar Dec 26 '23 10:12 Flexusma

I'll be giving the sign in screen a makeover in the coming days and would like to add some alternative login methods (auth token, QuickConnect). So if any of you want to start implementing some of that functionality, I might be able to use it for the redesign :)

Chaphasilor avatar Dec 28 '23 10:12 Chaphasilor

Okay, so Quick Connect is working now. Does anyone here use the jellyfin-plugin-sso, or something similar, and could help with testing?

Chaphasilor avatar Jan 10 '24 15:01 Chaphasilor

I'd love to help out with testing. Let me know what needs to be done.

9p4 avatar Jan 10 '24 16:01 9p4

If you're on Android, you just need to install the debug build below. It will install alongside the regular Finamp app, so you can keep using that without issues.

Download link (GitHub doesn't allow APK attachments): https://gofile.io/d/nAkfqF

Then just start the app and log into your server using Quick Connect :)

Alternatively, or if you're on iOS, you'd need to build the new-login-flow branch out yourself to test things.

Chaphasilor avatar Jan 10 '24 18:01 Chaphasilor

Seems like quick connect works (at least on Android).

9p4 avatar Jan 10 '24 21:01 9p4

Sweet, thanks! Is there anything else that you need or that would make sense for SSO? Anything that the Jellyfin app supports but Finamp does not?

Chaphasilor avatar Jan 10 '24 22:01 Chaphasilor

Quick connect is the recommended flow for using the SSO plugin on non-web-enabled apps/devices. As long as quick connect works fine, then it's good!

9p4 avatar Jan 10 '24 22:01 9p4

Thanks for implementing this, @Chaphasilor :)

xkcdstickfigure avatar Jan 11 '24 00:01 xkcdstickfigure

Download link (GitHub doesn't allow APK attachments): https://gofile.io/d/nAkfqF

This link has expired, but I'm eager to try the revamp - is there another dev build link, or should I just wait for the beta to release officially? I'm using jellyfin-sso, so the new version is somewhat necessary as 9p4 said.

TheRealGramdalf avatar Jan 23 '24 18:01 TheRealGramdalf

Also for reference see the issue about native client support

TheRealGramdalf avatar Jan 23 '24 20:01 TheRealGramdalf

You can wait for the official release or build the branch yourself :)

Regarding the client support issue you linked, that appears to make use of URL handlers, so if anyone has set the Jellyfin app as the default handler, it won't work with Finamp anymore. As long as no default is set, both should work (once this ends up in the server and we implemented support for it). But Quick Connect should work for the time being...

Chaphasilor avatar Jan 23 '24 21:01 Chaphasilor

Got it, many thanks. I'll have to get around to building it at some point. About native support I mainly meant it for "eventually" - definitely not part of the initial UI revamp release. Just something to put on your radar.

TheRealGramdalf avatar Jan 24 '24 02:01 TheRealGramdalf

@alx-alexpark @xkcdstickfigure @9p4 Quick connected support has now been released as part of the redesign beta. You can check it out here: https://github.com/jmshrv/finamp/releases/tag/0.9.2-beta

I'll close this issue for now since it should be resolved.

Chaphasilor avatar Feb 28 '24 14:02 Chaphasilor