-
-
Notifications
You must be signed in to change notification settings - Fork 347
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 |
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#1272 PR For application: You should update the extractor and prepare such a pr |
New PR to fix iOS player error, bot and http 403: TeamNewPipe/NewPipeExtractor#1272 |
…newpipe y ahora se usa rl repo oficial, estos cambios vienen de "Player fixes z-huang#1789", también se cambió el reproductor dndo una transparencia como fue solicitado por muchos
Hi @z-huang is this project still active? |
@gechoto hello There is such a popular app, but no one has added PoToken support. |
May i ask |
Some clients require PoToken, some require logins. The app automatically chooses the best one. You log in and continue. Does OuterTune use NewPipe extractor? Newpipe transmits iOS visitor data for bot errors. Each client uses different visitordata and potoken. PoToken fixes 403 errors |
It makes playback possible with the WEB_REMIX client in more/(most?) cases. Without po tokens this client only works for premium users.
Does playback work for you on the official website on the same device and mobile data connection in a private browser window?
Only small parts of it. Just like InnerTune with this PR.
Both InnerTune and OuterTune get visitor data in a different way than NewPipe.
Currently the same visitor data is used across all clients and po tokens are only supported for the WEB_REMIX client. |
So why don't you use iOS? It's pretty stable now. |
It is still in use as a fallback client but it does not work reliably for all users. |
IOS is not stable. Me and some users have problems with Mobile Data. Its still showing 'Log in to continue' most of the times in non age restricted videos. Under WIFI it works pretty stable. it needs log-in for age restricted videos tho. |
…ttling parameter of streaming urls" de "Player fixes z-huang#1789"
Btw @gechoto could you keep updating and add PoToken in this PR? |
I don't know yet since this is possibly going nowhere. For now I focus on OuterTune first because it is an active project where more people can enjoy the fixes. |
Thanks in advance, im trying to get it work in a private build from other project, but without success. |
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/13220889696/artifacts/2559580736