-
Notifications
You must be signed in to change notification settings - Fork 10.1k
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
Pasting specific text in multi-line quote causing that specific room to crash #6814
Comments
Reverted the multiline code block regular expression to the way it was before commit e2ea464. I believe this introduced the (potentially crippling) bug in Issue RocketChat#6814.
Confirmed this for sure brings down the browser.
And it immediately locks up the entire window. Also renders that channel completely useless until its removed directly from the db |
Can reproduce with as little as |
Traced it down to: https://github.com/RocketChat/Rocket.Chat/blob/develop/packages/rocketchat-markdown/markdowncode.js#L81 |
Created an issue: highlightjs/highlight.js#1505 |
Its been patched upstream: highlightjs/highlight.js@d05317a Just need to wait on a release or reference the commit. 👍 |
@geekgonecrazy We're still waiting on a release from highlight.js. How would we reference the commit? |
It looks like this was fixed in 9.12.0. How long till we see that reflected in RC? Will we have to wait for a RC update? |
@josephmarksjr i've updated my PR and will try and get it merged. Soon as its merged it'll be available in in develop. Then of course be available in the next release candidate and subsequent release. |
This is currently biting us (I think). One of our channels is not accessible. Is there any way to manually add the needed library update on an existing release? |
@tanc unfortunately its something that's compiled in on releases. Easiest way to correct this is to remove the message from the database unfortunately :( |
No problem @geekgonecrazy, I managed to use the REST API to delete the latest messages on that channel which freed it up and allowed us to connect again. Thanks for fixing this issue though and look forward to the next release! |
this is how we fixed it |
For any others that find this. Please do not blindly copy and paste the above. To fix the issue you would not need to drop oembed cache. You would have to find the offending message and prune it. using:
|
Just thought I'd comment - ran into this issue today and the above helped me resolve. My instance is running inside a docker container here are the specific steps I used:
Hope that helps someone! |
Thanks for sharing the tips! |
Rocket.Chat Version: v54
Running Instances: 1
DB Replicaset OpLog: Enabled
Runtime_Environment
Node Version: v4.5.0
Build_Environment
Node version: v4.7.3
I noticed an odd bug that can cripple any private or group chat. I copied and pasted a message someone sent, in order to sent it to someone else. I also wrapped it in ``` to form a multi-level quote.
Reproduce this by pasting the following test and wrapping it with ```:
I ....
When you send this, the chat page hangs and eventually you can't do anything. In Chrome it will eventually ask you to kill the page.
When you come back to RC (you'll have to manually go to a different url, such as a different dm or chat) and go back to the previous chat, it won't load or it won't let you send messages.
I was also able to reproduce this at http://demo.rocket.chat.
The text was updated successfully, but these errors were encountered: