firasdib / Regex101

This repository is currently only used for issue tracking for www.regex101.com
3.2k stars 198 forks source link

"A critical error has occurred" when positioning the cursor at a certain position (on "," symbol) in the regex #2216

Closed roship closed 5 months ago

roship commented 5 months ago

Bug Description

"A critical error has occurred" when positioning the cursor at a certain position in the regex.

Reproduction steps

"A critical error has occurred" when positioning the cursor at a certain position in the regex, exactly on "," symbol in REGEXP string.

REGEXP: (\d{4}-\d{2}-\d{2}_\d{2}:\d{2}:\d{2}.\d{5})\s+(\d{4}-\d{2}-\d{2}\s+\d{2}:\d{2}:\d{2}.\d{3})\s+(\w+)\s+(\d+)\s+---\s+[([\w-]+)]\s+([\w.]+)\s+:\s+(((Incident|Message)\s?(create|update|delete|received):\s?(Incident|InputMessage)?((id=([^,]+)?.)?))?.)

Expected Outcome

Browser

Chrome Version 120.0.6099.234 (Official Build) (arm64)

OS

macOS Sonoma Version 14.2.1 (23C71)

working-name commented 5 months ago

Hello @roship I downloaded the same chrome version as yours, and have grammarly and 1password extensions. I could not make it crash. Also, the screenshot regex is not the same as the pasted one, but either way, while editing it and clicking around, including on the , character, I did not manage to make it crash.

Would you try to reproduce it on your end, maybe without addons enabled?

roship commented 5 months ago

Yep, works fine without addons (incognito mode). I should have thought about it. My apologies for all your extra work because of this.

The cause of trouble was addon "Burning Vocabulary"

working-name commented 5 months ago

Awesome, glad you found the culprit!

roship commented 5 months ago

I have to say your service is the best I've found.

working-name commented 5 months ago

Agreed, I use the app quite a bit myself. I like it. @firasdib did a great job 😄