-
Notifications
You must be signed in to change notification settings - Fork 811
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
Invalid regular expression: missing / #1110
Comments
I'm having the same problem as well. Receiving the same error as above. Everything worked fine yesterday. |
Experiencing same issue, getting identical stack trace as well. |
Okay I am trying to figure out the problem but this is my first time working with the source code of this project and with the youtube cipher algorithm so it's taking a while anyways I've narrowed the problem down to a problem with evaluating nTransformScript because the code returned from youtube is for some reason broken, that's the conclusion I reached |
Still having same problem |
this solved it for me |
If you need to use it rn and don't want to/can't tinker inside module |
|
Not sure if that's the proper solution or not but it does work |
Can reproduce: with almost all videos that I test with:
|
same problem with any video
|
@austinharms deleted my node modules I'm back! Holy shit I was about to wipe my system because this week I had some pc malware issues I solved. But when in doubt CHECK the GITHUB! I was that close....thanks to everyone for the comments... I skew front-end but hoping devs solve the issue! |
But if we know 4.10.0 works, then we know that the PR that came after introduced a bug....so whats the PR for 4.10.1? |
@dsnsgithub on 4.10.0 its slow but downloading but will test audio when its up |
@dskaiser82 Thank you, your fix works. ytdl-core@4.10.0 seems to be working while ytdl-core@4.11.0 seems to have a problem with dependences. |
@dsnsgithub Good! Yean I have audio working fine too on the video below....but did take a longer than normal....I want to test another vid to see if speeds can be faster....but for now I'll take this band aid fix |
Yeah I think this version is slower than before, but I do like to do highest Audio, Highest Video |
Last week it was closing braces in strings. To confuse parsers. |
Can anyone please attach base.js file that has this issue because I can't reproduce this on my end. |
@WaqasIbrahim sent via email |
I've got the same issue +1 |
Needs exports.cutAfterJSON in util.js to skip over regex. When it encounters /,,[/,913,/](,)}/ as an array value in base.js it sees the } as the end of the n code function that is being parsed. So evaluation breaks. |
I just tested ncode extraction on a file @gatecrasher777 sent. The solution I suggested in the closed issue: #1108 (comment) still works. |
this fix is working for me 100% |
downgrading the version decreases the singnificat download spped, how can i solve that ? |
For some reason, I kept getting |
@arinmodi Thats the state I chose, degraded version but MUCH slower download speeds.....no PR's in yet for the fix right? |
Made a fork of the temporary fix here: https://github.com/GreepTheSheep/node-ytdl-core You can add it on your package.json with npm |
@GreepTheSheep PR waiting for approval....anyone know who has PR approval/merge rights? |
same problem with any videos, I have the latest version of ytdl-core |
|
downgrading to 4.10.0 solved the problem for me! Waiting for a new version... |
Let me know if I can throw a few bucks the maintainers' way, they're doing God's work. |
This error message just sudenly changed... yesterday it was the invalid regex, now it's something new: evalmachine.<anonymous>:24
c[72],c[6]),(0,c[85])(c[11]),(0,c[41])(c[39],c[28],(0,c[66])()),(0,c[78])(c[Math.pow(6,2)+122+-119],c[new Date("Wednesday December 31 1969 19:00:1
4 CDT")/1E3])}catch(d){(0<c[0]||(((0,c[80])(c[60],c[88]),c[80])(c[21],c[88]),0))&&(0,c[17])((0,c[70])(c[6]),c[78],c[88],c[12])}try{2<c[91]&&(1<c[5
3]?(0,c[17])((0,c[80])(c[82],c[6]),c[78],c[39],c[5]):((0,c[45])(c[49],c[88]),c[78])(c[88],c[86])),(0,c[17])((0,c[85])(c[31]),c[41],c[88],c[75],(0,
c[35])())}catch(d){}};Wka(ncode);
^
SyntaxError: Missing catch or finally after try
at new Script (node:vm:100:7)
at Object.exports.decipherFormats (D:\JS\revolt butler\node_modules\ytdl-core\lib\sig.js:116:51)
at runMicrotasks (<anonymous>)
at processTicksAndRejections (node:internal/process/task_queues:96:5)
at async Promise.all (index 0)
at async exports.getInfo (D:\JS\revolt butler\node_modules\ytdl-core\lib\info.js:401:17) |
PR: #1111 We have some approvals but this is the blocker: Who is a maintainer? |
Wanted to add that this can too be fixed by downgrading to 4.10.0 |
Downgrading to 4.10.0 will mean the N transform gracefully fails. Everything will work, but downloads will be throttled. |
🎉 This issue has been resolved in version 4.11.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
They're at it again, it stopped working just today. |
Today I'm get the same error again |
|
Same here receiving since about yesterday 10pm UK time:
|
Found a fix: #1111 Same solution as the previous regex errors. |
yes the error come's back |
and downgrading to 4.10.0 solved the problem, again |
Is it throttled though? |
Since 1 or 2 hours, all videos passed on ytdl-core return this "Invalid regular expression" error
Stack trace:
ytdl-core version: 4.11.0
Edit: Temporary fix linked here: #1110 (comment)
The text was updated successfully, but these errors were encountered: