[Live Markdown Issues] Android - Chat - App isn't responding on pasting email repeatedly in compose box

[lanitochka17]

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!

---

Version Number: 1.4.90
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:

Action Performed:

1. Launch app
2. Tap on 1: 1 report of expensifail user eg: [email protected]
3. Tap header
4. Copy email
5. Navigate to 1:1 page
6. Repeatedly paste email in compose

Expected Result:

From header copy email and paste in compose repeatedly, app isn't responding message must note be displayed

Actual Result:

From header copy email and paste in compose repeatedly, app isn't responding message is displayed

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

• Android: Native
• Android: mWeb Chrome
• iOS: Native
• iOS: mWeb Safari
• MacOS: Chrome / Safari
• MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

Bug6376901_1707757873412.not.mp4

logs (2).txt

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~01c622c8bf192a870b
• Upwork Job ID: 1757099852323344384
• Last Price Increase: 2024-02-12
• Automatic offers:
• aimane-chnaif | Reviewer | 0

[melvin-bot]

Job added to Upwork: https://www.upwork.com/jobs/~01c622c8bf192a870b

[melvin-bot]

Triggered auto assignment to Contributor-plus team member for initial proposal review - @aimane-chnaif (External)

[melvin-bot]

Triggered auto assignment to @mallenexpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

[lanitochka17]

We think that this bug might be related to #vip-vsp
CC @quinthar

[nrajyaguru]

When user paste the text
There is change method getting invoked further which has series of actions internally and finally lead to no class found exception. Let me know if this is okay to go ahead
CC: @mallenexpensify @lanitochka17

[melvin-bot]

📣 @nrajyaguru! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:

1. Make sure you've read and understood the contributing guidelines.
2. Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
3. Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
4. Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.
   
   Format:

Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

[nayabatir1]

Hey @lanitochka17,
Is this issue specifically related to expensifail user?
If so how to create one? I've tested locally with normal user and I'm unable to reproduce.

[melvin-bot]

📣 @nayabatir1! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:

1. Make sure you've read and understood the contributing guidelines.
2. Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
3. Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
4. Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.
   
   Format:

Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

[mallenexpensify]

Unable to reproduce on iOS, @aimane-chnaif can you test on android? Thx

[aimane-chnaif]

reproduced on android

On 5th repeat, it took 10s to be pasted into composer. Seems related to live markdown

[aimane-chnaif]

After copy pasting email several times, composer is super slow when try to edit message and almost unusable.
Huge performance lag

[mallenexpensify]

Checking on in the markdown room https://expensify.slack.com/archives/C06BDSWLDPB/p1707866403183029
Thanks Aimane.

[tomekzaw]

Thanks for reporting this issue, I will investigate it, please assign me.

@thienlnam Could we also add this issue in #36071?

[melvin-bot]

📣 @aimane-chnaif 🎉 An offer has been automatically sent to your Upwork account for the Reviewer role 🎉 Thanks for contributing to the Expensify app!

Offer link
Upwork job

[melvin-bot]

@tomekzaw, @mallenexpensify, @thienlnam, @aimane-chnaif Whoops! This issue is 2 days overdue. Let's get this updated quick!

[tomekzaw]

Working on it

[SzymczakJ]

Still working at it. Current status: trying to profile Hermes runtime to find out which part of ExpensiMark parser is working too slow. With that knowledge maybe we will be able to make it faster.

[SzymczakJ]

Current update: investigated performance of ExpensiMark on Hermes. The input for profiling ExpensiMark was "[email protected][email protected][email protected][email protected]" which is basically and email from this issue reproduction multiplied 4 times.

As we can see execution of regExpPrototypeExec takes more than 2 seconds, so we can be sure that the RegExp implementation for Hermes is the problem here. Digging deeper into Hermes internals to further investigate the problem.

[mallenexpensify]

@SzymczakJ is the bug reproducible with other text or data being pasted repeatedly in the compose box? I want to add this to a VIP/wave project and I'm thinking #vip-vsb. Wanting to know if small business users will be affected by this or if it's an edge case that only happens with pasted emails.

[SzymczakJ]

@mallenexpensify I didn't manage to reproduce it with anything other than emails.
I've also measured time of Expensimark parsing which is a bottleneck in this issue, these are result for pretty small inputs:
[email protected] - 4ms
[email protected]@somedomain.com - 5595ms!
[email protected] - 1ms
[email protected]@testDomain.com - 7ms
[email protected] - 4ms
[email protected]@testDomain.com -5405ms!
Keep in mind that we want to keep parse time at the maximum of 3 ms for the experience to be smooth, so even parsing single mail like [email protected] takes too long.

[mallenexpensify]

Thanks @SzymczakJ , appreciate the context, I had no idea that it took longer to parse emails with more characters.

@thienlnam , lil help here. with this only affecting users who paste a bunch of emails in at once, do you think it's worth fixing right now or should we close then maybe revisit later?

One reason TO potentially fix now would be if email parsing updates also benefited or sped up other places in the app. (I have no clue)

[SzymczakJ]

@mallenexpensify after spending some time on this issue these are my thoughts:

• it's not worth to spend so much time on such edge case that doesn't happen when using app without trying to purposefully break it
• even if we fix this edge case there will be other 'attack strings' that also cause problems for our regexes and more such problems may arise

Maybe we should think of some more general and long-term solution for this "live-markdown working slow with some inputs" issue. Do you have other ideas?
cc @roryabraham @thienlnam

[melvin-bot]

@tomekzaw @mallenexpensify @thienlnam @SzymczakJ @aimane-chnaif this issue is now 4 weeks old and preventing us from maintaining WAQ. This should now be your highest priority. Please post below what your plan is to get a PR in review ASAP. Thanks!

[melvin-bot]

Current assignee @aimane-chnaif is eligible for the Internal assigner, not assigning anyone new.

[thienlnam]

Yeah I don't think this is an urgent bug we need to fix - it's definitely a malicious edge case so we can add it to the list but can prioritize other issues before this

[thienlnam]

I added this to the main tracking issue #36071, and adjusted priority

[mallenexpensify]

Added to #vip-vsb because the bug doesn't have to do with money. I wonder, since there's also the tracking issue, if there's another/better way to manage this ¯_(ツ)_/¯

• HIGH: [Live Markdown] Tracking Live Markdown Issues #36071

[mallenexpensify]

It looks like Jack and gang are going through issues, from high > low and this one is medium.

• HIGH: [Live Markdown] Tracking Live Markdown Issues #36071

[mallenexpensify]

#36071 was closed

Closing this! We've worked through all the critical issues, I think every issue from here on out can just be addressed independently. We're still working through #39518 but there's no longer a need for this main tracking issue

So.. now... do we want to do anything here? Fix it? add retest-weekly label? Start a discussion in the #react-native-live-markdown?

From @SzymczakJ above

after spending some time on this issue these are my thoughts:

• it's not worth to spend so much time on such edge case that doesn't happen when using app without trying to purposefully break it
• even if we fix this edge case there will be other 'attack strings' that also cause problems for our regexes and more such problems may arise

Maybe we should think of some more general and long-term solution for this "live-markdown working slow with some inputs" issue. Do you have other ideas?

[thienlnam]

I think we can close for now and re-open if this comes up again. Realistically this doesn't seem like a common use-case and the time tradeoff does not seem worth it