-
Notifications
You must be signed in to change notification settings - Fork 3.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
[Performance] Loading chats takes 10+ minutes for guides and can often fail altogether #21444
Comments
Triggered auto assignment to @arosiclair ( |
I need to investigate why the reauthentication mechanism did not fire and got stuck in an infinite loop retrying over and over. |
interesting that's Unauthorized invalid token and not token expired... 🤔 |
Here's the app logs https://www.expensify.com/_devportal/tools/logSearch/#query=request_id:(%227dbf356c7ab21f9d-ATL%22)+AND+timestamp:[2023-06-23T18:41:28.436Z+TO+2023-06-23T20:41:28.436Z]&index=logs_expensify-022949 I don't get why I don't see any calls to reauthenticate, I tried locally and I do see them |
Wait actually the requests stuck are like this https://www.expensify.com/_devportal/tools/logSearch/#query=request_id:(%227dbf35385fc91f9d-ATL%22)+AND+timestamp:[2023-06-23T18:41:20.104Z+TO+2023-06-23T20:41:20.104Z]&index=logs_expensify-022949
Revoked... still it's 407 |
Code is here which means the authToken was issued before we updated |
Could not find that authToken being set for that account in the logs https://www.expensify.com/_devportal/tools/logSearch/#query=blob%3A%22Invalidating%20authTokens%20by%20setting%20PRIVATE_MINIMUM_AUTH_TOKEN_ISSUE_TIME%20for%20accountID%3A%2011736036%22%20AND%20timestamp%3A%5B2023-06-20T00%3A00%20TO%202023-06-24T23%3A59%5D&index=_all |
Going to close this to have a clean slate for the issue so we don't distract new people with my previous investigations... |
New issue here #21766 |
Can I get compensation here for reviewing linked PR? |
Hi @aimane-chnaif , would this be $250? Also do you receive payment in newdot or in upwork? |
@kevinksullivan I am not sure. We agreed all internal review compensations are flatten 1k afaik. Easy PR will cover for big PR. |
@aimane-chnaif thanks, where have you been receiving payment for recent work? NewDot or Upwork? |
upwork |
offer sent @aimane-chnaif https://www.upwork.com/jobs/~010ec101896471dba7 lmk when you accept! |
Accepted, thanks! |
All set |
Slack https://expensify.slack.com/archives/C036S0BVAHH/p1687540879337529
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
What performance issue do we need to solve?
Guides (who have a lot of policies and reports) are often unable to move between chats without NewDot showing a blank chat screen and loading for minutes (or failing to load altogether.
What is the impact of this on end-users?
Guides cannot load chats and talk with their assigned leads.
List any benchmarks that show the severity of the issue
Please also provide exact steps taken to collect metrics above if any so we can independently verify the results.
Proposed solution (if any)
No idea
List any benchmarks after implementing the changes to show impacts of the proposed solution (if any)
Note: These should be the same as the benchmarks collected before any changes.
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: v1.3.30-5
Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers): christopher.bowers@team.expensify.com , natasha.varner@team.expensify.com
Notes/Photos/Videos: Any additional supporting documentation
Videos / screenshots in the slack link examples above
View all open jobs on Upwork
The text was updated successfully, but these errors were encountered: