badeggg

Results 21 comments of badeggg

@IuliiaHerets Can you please comment the exact text that was pasted inside the compose box?

Edited by **proposal-police**: This proposal was **edited** at 2024-11-21 07:37:15 UTC. ## Proposal ### Please re-state the problem that we are trying to solve in this issue. App stalls when...

> App still gets stuck. Yeah, it's frustrating. I noticed the size of the regexp is not the cause. I guess it's because some part of the regexp 'somehow' trigger...

I have updated [my proposal](https://github.com/Expensify/App/issues/52475#issuecomment-2478497680)

I have updated [my proposal](https://github.com/Expensify/App/issues/52475#issuecomment-2478497680), with a more accurate root cause.

It's a pattern in expensify-common, part of [regexp for video](https://github.com/Expensify/expensify-common/blob/302e37e40d9d7e718e7c1c3d98faaa644b51da9d/lib/ExpensiMark.ts#L79-L82) match this pattern. I will point out it latter, I'm kind busy now

I have updated [my proposal](https://github.com/Expensify/App/issues/52475#issuecomment-2478497680). Thank you for your patience. I also updated my opinion. The root cause is **regexp execution get stuck when too many possibilities before failing**, pattern...

The alternative solution in my proposal can solve this problem, change the super complicated url pattern to a simple url pattern like: `https?:\/\/[0-9a-z\.-\/?=&#]+`. But it's a big change, guess this...

Hi, I have updated [my proposal](https://github.com/Expensify/App/issues/52475#issuecomment-2478497680), just added tech details for the alternative solution.

- Guess my description "regexp execution get stuck when too many possibilities before failing" has a formal name: catastrophic backtracking. - I looked into the regex, the reason for catastrophic...