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

[2014 Player] Videos watched with 2014 player aren't updating watch history #493

Open
i-lost-my-bagel opened this issue Jan 20, 2024 · 6 comments
Labels
bug Something isn't working classic player This issue affects one or more classic player options. It does not affect the latest player. experimental feature

Comments

@i-lost-my-bagel
Copy link

i-lost-my-bagel commented Jan 20, 2024

Rehike version

0.8

Operating system

Linux 6.1.0-17-amd64 x86_64

PHP version

PHP 8.2.7

What's going on?

If I have the experimental 2014 player set then any videos watched are not added to my watch history

@i-lost-my-bagel i-lost-my-bagel added the bug Something isn't working label Jan 20, 2024
@OleExploder
Copy link

OleExploder commented Jan 21, 2024

Same thing happened to me but in Windows 11 22H2 and when using PHP 8.2.12.

@TorutheRedFox
Copy link
Contributor

TorutheRedFox commented Jan 25, 2024

accidental privacy W?

jokes aside the fix should only be enabled if watch history is enabled too

@olioliluaaa
Copy link

Same issue with me, Windows 10 Pro 22H2 with PHP 8.2.12, on the classic 2015 player. On the latest player it updates as usual though.

@kawapure kawapure added the classic player This issue affects one or more classic player options. It does not affect the latest player. label Jun 8, 2024
@kawapure
Copy link
Member

kawapure commented Jun 8, 2024

Same issue with me, Windows 10 Pro 22H2 with PHP 8.2.12, on the classic 2015 player. On the latest player it updates as usual though.

It affects everyone. The player itself simply doesn't make the requests necessary to log the watch history correctly.

@TorutheRedFox
Copy link
Contributor

Same issue with me, Windows 10 Pro 22H2 with PHP 8.2.12, on the classic 2015 player. On the latest player it updates as usual though.

It affects everyone. The player itself simply doesn't make the requests necessary to log the watch history correctly.

actually thinking about it, i think we patched those requests out because they were causing a self-DoS as the actual endpoint itself wasn't implemented and falling back to trying to load a channel page

@kawapure
Copy link
Member

Same issue with me, Windows 10 Pro 22H2 with PHP 8.2.12, on the classic 2015 player. On the latest player it updates as usual though.

It affects everyone. The player itself simply doesn't make the requests necessary to log the watch history correctly.

actually thinking about it, i think we patched those requests out because they were causing a self-DoS as the actual endpoint itself wasn't implemented and falling back to trying to load a channel page

No, those were generate_204 requests which were optimised to be ignored by the server. As far as I'm aware, generate_204 has no effect on behaviour at all.

Watch history updates are controlled by a separate request (/api/stats/playback) which is requested by modern player + in Rehike's PHP code manually here, since we needed to populate watch history for signed out sessions to have homepage recommendations. It's just an oversight in the 2014 player that prevents it from making these requests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working classic player This issue affects one or more classic player options. It does not affect the latest player. experimental feature
Projects
None yet
Development

No branches or pull requests

6 participants