-
-
Notifications
You must be signed in to change notification settings - Fork 334
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
Player fixes #1789
base: dev
Are you sure you want to change the base?
Player fixes #1789
Conversation
️✅ There are no secrets present in this pull request anymore.If these secrets were true positive and are still valid, we highly recommend you to revoke them. 🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request. |
@z-huang can you have a look? There is one last thing I'm not sure about. The http clients in What do you think? Should the proxy be removed? |
The crashing after logging in seems to be fixed in the latest commit build! (However, the youtube music webpage is opened after logging in successfully, it would maybe be better if it exited out of the login prompt automatically, after logging in successfully) EDIT: Playing songs, as well as downloading, also works of course! |
Works great! Only issue is increased startup time and decreased UI performance on a lower end device like mine (Cat S22 Flip). It takes about 10-15 seconds for my phone to start it up, and then the UI renders slowly (5-10 FPS when scrolling) |
@MLGHerobrine if you used the link to download the debug version, that is normal. I haven't tested but I assume that when building the release APK, performance should be as usual. |
Proxy support is now also added. Turns out it wasn't hard because the proxy is only applied on app start. This means if you change the proxy settings you will have to fully close and restart the app. That was already the case before and just made adding the proxy to the new http clients easier. |
This video won't play or download when you sign in. I have selected "high" for audio quality and it shows mp4 in the details: |
@Figim Please follow these steps: 1.) Clear app data (full reset without restoring a backup)
This PR does not change anything about the code which chooses the format. "high" in InnerTune just means the format with the highest bitrate (for you). This could be improved in the future but for now you get the same formats as with older InnerTune versions. |
2. Playing this video |
Broken Today: got Response code: 403 while playing AgeRestricted Content (It was working before). |
Can confirm, I couldn't listen to the songs (not age-restricted ones) while riding on train. It gives error after 30 seconds of listening. |
This is unrelated to this PR. Open a separate issue.
You can open an issue without logs in this case since it seems that...
...other people can reproduce it.
you don't need that - adb works perfectly fine without android studio |
@th3y Do you have a chance to attach a debugger and look at the response of the player request with the WEB_CREATOR client?
@KananHasanov747 The "while riding on train" part might be the issue. Sometimes even official clients refuse to work when on public wifi or mobile network. If you aren't already you can try logging in. If you are using mobile network you can try to enable/disable airplane mode and restart the app (make sure it is fully stopped and not running in background anymore before starting again). |
Error 403 (Logged in only) so its about newpipe issue |
@th3y how have you tested "Logged in only"? Can you test fresh install logged in vs fresh install logged out? |
Works logged out, using another client than WEB_CREATOR gives error 403. |
@th3y please test the TVHTML5 client |
You are a hero, using TVHTML5 works! (Logged in and age restricted) |
While TVHTML5 works, there are some urls that returns 403 (Most of them are music videos or from video section) |
|
• Does it return 403 consistently on the same videos or do they work if you close the app and try again a few minutes later? |
If it doesn't even work with the official client in a browser your IP or account might be blocked.
How often does this happen? |
I can give you some ids to test (I think i should not share yt links) Its not about the owner channel, i think it could be related from the 'embed' or something behind it, they are managed in a different way |
Im sorry, typo error, it works with official client. IOS gives 30 seg error or LOGIN_REQUIRED (Confirm you are not a bot) sometimes. Or it plays at 30 seg and then error. About the PR, i can't test it right now |
Today i migrated the code to kotlin, and with mobile data i didn't get any playback error with IOS (Of course age restricted videos are not working) Edit: |
… de gechoto hasta remove unused PipedResponse model
@th3y Was this in NewPipe or InnerTune? Is there a difference between the two for you? Edit: |
Im using NewPipe for login only. (The last comment was about IOS client) |
Playback OK without problem or skips (Using your commit too) |
NewPipe is preparing a new PR with a new potoken version of Extractor that will be released soon to fix bot and http 403 errors. PR for extractor: TeamNewPipe/NewPipeExtractor#1247 (review) PR For application: You should update the extractor and prepare such a pr |
Should fix #1748 #1781 #1775 #1770 #1758 #1764 #1760 #1757 #1787 #1788 #1790
This supersedes #1774 with additional improvements.
Compared to the previous PR this adds:
songUrlCache
Current test build:
https://github.com/z-huang/InnerTune/actions/runs/12946584020/artifacts/2479712770