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

[Bug] Cannot Login #91

Closed
soulspark666 opened this issue Jul 22, 2023 · 3 comments
Closed

[Bug] Cannot Login #91

soulspark666 opened this issue Jul 22, 2023 · 3 comments
Labels

Comments

@soulspark666
Copy link

Describe the bug
Cannot log in in the App, tried bot manual and in-app login methog

Environment

  • OS name and version: Android 13
  • SpMp version (include commit hash if you are using a non-release build) : 0.1.3(5)
  • Accounts (were you signed in to YouTube, Discord, etc. in the app?): No

Screenshots
Manual Login:
photo1690001277

Logs

java.lang.NullPointerException
 at com.toasterofbread.spmp.model.YoutubeMusicAuthInfo$Companion$fromHeaders$2.invokeSuspend(Unknown Source:212)
 at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(Unknown Source:8)
 at kotlinx.coroutines.DispatchedTask.run(Unknown Source:100)
 at kotlinx.coroutines.internal.LimitedDispatcher.run(Unknown Source:12)
 at kotlinx.coroutines.scheduling.TaskImpl.run(Unknown Source:2)
 at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run(Unknown Source:79)
 Suppressed: coil.network.HttpException: [androidx.compose.ui.platform.MotionDurationScaleImpl@17c2e37, androidx.compose.runtime.BroadcastFrameClock@d7e9fa4, StandaloneCoroutine{Cancelling}@66210d, AndroidUiDispatcher@69c38c2]
@toasterofbread
Copy link
Owner

Should be fixed now, please let me know if it works.

I believe the issue was due to the account being used not being initialised as a YouTube music channel (which is required due to a technical limitation in the app right now), so I was only able to test it properly a few times since I only have so many Google accounts. The account registration dialog I've added will be made optional once I've fixed the limitation I mentioned.

@soulspark666
Copy link
Author

soulspark666 commented Sep 10, 2023

Hi @toasterofbread , sorry for the late reply, I may have missed your response.

SpMp version: 0.1.4(6) [Clean Install]

Login Manually:

Screenshot_20230910-154104_SpMp

Login Google sign in:

Screenshot_20230910-152634_SpMp
And it crashed.

@toasterofbread
Copy link
Owner

toasterofbread commented Sep 10, 2023

This fix isn't in 0.1.4, you'll have to try a recent build or wait for the next release, which I'm planning to publish in a day or two.

toasterofbread added a commit that referenced this issue Nov 12, 2023
Not really necessary, but added due to a current technical limitation.
Will be made optional in the future.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants