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

Invalid regular expression: missing / #774

Closed
unixbird opened this issue Aug 18, 2022 · 22 comments
Closed

Invalid regular expression: missing / #774

unixbird opened this issue Aug 18, 2022 · 22 comments
Labels

Comments

@unixbird
Copy link

What's happening?
When trying to do a normal /play query: whatever command the bot joins and then the message "ope: Invalid regular expression: missing /" pops up. The bot will then stay for 30 seconds and then leave. No crash messages are present

Message #music

Logs
Looking at docker logs gives normal startup and last message is the discord invite link.

Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
Add any other context about the problem here.

Runtime
I'm running Muse:
Docker

Versions

  • Muse: Latest
  • Docker (if applicable): 20.10.17
  • OS: Ubuntu 20.04
@sypion
Copy link

sypion commented Aug 18, 2022

Here's a screenshot too if necessary:
image

@Dumdidldum
Copy link

facing the same issue here

@Maikel1990
Copy link

Also having the same issue

@Midefos
Copy link

Midefos commented Aug 18, 2022

Hello, I am having the same issue here:
imagen
Searching by text in the /play command works well, but when playing something, it appears.

@gecko10000
Copy link

The full error looks like this (I added console.log(error) here):

evalmachine.<anonymous>:11
"mfZru","cfYAzm",1897615910,414154674,"IluoVc5",null,1,"XsmT",-659785013,577478260,-419211961,1087899650,/,,[/,913,/](,)};Wka(ncode);
                                                                                                         ^

SyntaxError: Invalid regular expression: missing /
    at new Script (node:vm:101:7)
    at exports.decipherFormats (/home/workspace/muse-eternals/node_modules/ytdl-core/lib/sig.js:116:51)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async Promise.all (index 0)
    at async exports.getInfo (/home/workspace/muse-eternals/node_modules/ytdl-core/lib/info.js:401:17)

@xMikux
Copy link

xMikux commented Aug 19, 2022

Look like it is from ytdl-core issue: fent/node-ytdl-core#1110

There have a temporary fix, and I try to change the package.json, build again, and it back to work normally.
fent/node-ytdl-core#1110 (comment)

@codetheweb
Copy link
Collaborator

I tried downgrading ytdl-core to 4.10.0 and it still didn't work for me. If someone lets me know when the issue is resolved on ytdl-core I can bump the version here as needed (PRs are also great :)).

@brianterry-ama
Copy link

Switching to the suggested branch for fixed it for me
npm i ytdl-core@https://github.com/GreepTheSheep/node-ytdl-core

@xMikux
Copy link

xMikux commented Aug 20, 2022

Maybe we can build the one temporary fix image for now, until they merge the fix.

@unixbird
Copy link
Author

unixbird commented Aug 20, 2022

So it looks like the docker image was updated. The error is still the same but has changed itself to say "Missing catch or finally after try" on my end and previous messages with the original "Invalid regular expression: missing /".

@xMikux
Copy link

xMikux commented Aug 20, 2022

yeah, repo author try to downgrade ytdl-core, but it doesn't fix.

@xMikux
Copy link

xMikux commented Aug 20, 2022

I will build the temporary fix image later, until ytdl-core merge the fix pr, and bump up version.

@xMikux
Copy link

xMikux commented Aug 20, 2022

Okay done.

Image (will delete when this issue fix):
DELETED

Commit and action:
xMikux@1bf9f46

@xMikux
Copy link

xMikux commented Aug 27, 2022

It looks like this issue has been fixed.

@davidchalifoux
Copy link

@codetheweb Hey, I just pulled the latest docker image and got this error. Should the latest be fixed?

@mysticpoisen
Copy link

mysticpoisen commented Sep 1, 2022

I've also been seeing this with both latest, 2.1.6, and xMikux's fix package. Was working fine yesterday. If this is still just yt-dl issues, I might have to look into a non-docker install so I can update it myself as it breaks.

@unixbird
Copy link
Author

unixbird commented Sep 1, 2022

I remember reading how youtube was doing some things to make ytdl less easy to use. But this is happening to me as well again.

@codetheweb
Copy link
Collaborator

fent/node-ytdl-core#1131

@xMikux
Copy link

xMikux commented Sep 1, 2022

Another temporary fix on fent/node-ytdl-core#1131 (comment)

I will build fix image again, but not sure what will cause actually (I using sed command to replace string)
but it look like can normally play music.

@xMikux
Copy link

xMikux commented Sep 1, 2022

Okay another done.

Image (will delete when this issue fix again):
DELETED

Commit and action:
xMikux@e58ec02

@mysticpoisen
Copy link

mysticpoisen commented Sep 1, 2022

Confirming @xMikux's fix package is working(and giving me nice english text too). Thanks!

@xMikux
Copy link

xMikux commented Sep 4, 2022

Well, I just give up my weird english.

And I see there have a new release ytdl-core v4.11.2 fix this again.

Hope it won't break again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests