Implement spotify token from configuration

fiso64 / slsk-batchdl

A batch downloader for Soulseek

GNU General Public License v3.0

216 stars 16 forks source link

Implement spotify token from configuration #53

Closed FoxxMD closed 1 month ago

FoxxMD commented 1 month ago

Makes headless usage of sldl easier #42

Adds --spotify-token arg for users to provide their own token.
If token is provided oauth flow doesn't run and browser does not need to open
If oauth flow does run then token is logged to console

Allows a user to generate a token by running sldl locally and then use that token with a headless instance.

fiso64 commented 1 month ago

Thank you. I'm in the middle of a large refactor and didn't push (I wasn't expecting any contributions). I can push it so that you can adapt your pr, or I can close the pr and then reimplement this myself in the new code. What do you prefer? (I'm unsure which would be best, as I don't have any experience with dealing with contributions)

FoxxMD commented 1 month ago

However you want to get the newer code out is up to you and I'll refactor this (and the other PR) to your changes once the code is available.

No rush to push the new code out either. I've got this and the other PR working on my fork in the foxx branch and foxxmd/slsk-batch:foxx docker image tag so I'm functionally fine for the time being.

fiso64 commented 1 month ago

I pushed it, you can now modify the two PRs. Configuration options have been moved to Config.cs, and help printing is now done in Help.cs. Aside from that there are many optimizations and bug fixes in this commit, although it may be less stable as I haven't done much testing.