tehkillerbee / mopidy-tidal

Tidal Backend plugin for Mopidy
Apache License 2.0
91 stars 28 forks source link

[Question] Authentication flow #176

Open GioF71 opened 2 weeks ago

GioF71 commented 2 weeks ago

Hello, with referent to this issue, can we use the regular oauth2 auth flow and get HI_RES_LOSSLESS quality on mopidy-tidal as well?

tehkillerbee commented 2 weeks ago

Yes, I believe that should work as well but I have yet to test it.

fmarzocca commented 2 weeks ago

What's the point not using pkce?

GioF71 commented 2 weeks ago

What's the point not using pkce?

AFAIK currently you cannot get new authentication tokens, at least not with the known workflow. I personally get an error when I try that.

GioF71 commented 2 weeks ago

What's the point not using pkce?

and also it would be better if we can avoid the complexity of the process. The regular flow is simpler

tehkillerbee commented 2 weeks ago

What's the point not using pkce?

AFAIK currently you cannot get new authentication tokens, at least not with the known workflow. I personally get an error when I try that.

Can you elaborate the error? PKCE still works when using python-tidal directly.

GioF71 commented 2 weeks ago

What's the point not using pkce?

AFAIK currently you cannot get new authentication tokens, at least not with the known workflow. I personally get an error when I try that.

Can you elaborate the error? PKCE still works when using python-tidal directly.

with your existing token, yes, it also works for me. But have you tried to get a new one? Here's when the error happens for me.

tehkillerbee commented 2 weeks ago

What's the point not using pkce?

AFAIK currently you cannot get new authentication tokens, at least not with the known workflow. I personally get an error when I try that.

Can you elaborate the error? PKCE still works when using python-tidal directly.

with your existing token, yes, it also works for me. But have you tried to get a new one? Here's when the error happens for me.

Just tried (using the pkce_example in python-tidal) creating a new token. It still appears to work as expected. Are you trying to authenticate through mopidy-tidal?

GioF71 commented 2 weeks ago

What's the point not using pkce?

AFAIK currently you cannot get new authentication tokens, at least not with the known workflow. I personally get an error when I try that.

Can you elaborate the error? PKCE still works when using python-tidal directly.

with your existing token, yes, it also works for me. But have you tried to get a new one? Here's when the error happens for me.

Just tried (using the pkce_example in python-tidal) creating a new token. It still appears to work as expected. Are you trying to authenticate through mopidy-tidal?

Alright, in fact today it works again. yesterday I was getting an error exactly when I had to enter the username on the form. Ok, so this problem is apparently not here anymore.