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

JavaScript interpreter error #48

Closed
scoliono opened this issue Dec 19, 2024 · 4 comments · Fixed by #49
Closed

JavaScript interpreter error #48

scoliono opened this issue Dec 19, 2024 · 4 comments · Fixed by #49
Labels
bug Something isn't working

Comments

@scoliono
Copy link

scoliono commented Dec 19, 2024

This is not a duplicate of #44, I've already applied the fix for that. This just started happening to me this evening, and at the same time I'm now getting random 403s with /videoplayback.

inv_sig_helper-1  | [2024-12-19T02:26:24Z ERROR inv_sig_helper_rust::jobs] JavaScript interpreter error (sig code): Some(Exception { message: Some("unexpected token in expression: '}'"), file: Some("eval_script"), line: Some(1), column: Some(215), stack: Some("    at eval_script:1:215\n") })

I'm glad to hear we will be getting Invidious Companion soon 😅

@alive4ever
Copy link
Contributor

Hi, I somehow managed to fix the signature extraction.

First, revert to this commit , since the last commits changes the algorithm too much.

git reset ef11b83a && git restore .

Then apply this fix (i.e. git apply fixup-inv-sig-helper.patch)

fixup-inv-sig-helper.patch

Tested with player version 03dbdfad.

@GTechAlpha
Copy link

Here's a minimal patch for this in the mean time:

https://0x0.st/XCDZ.patch

@squatica
Copy link

Hmm, why not a PR?

@stryk3rr3al
Copy link

+1 for PR and following this issue.

@unixfox unixfox added the bug Something isn't working label Dec 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants