-
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
An 'Unexpected Error' occurs when approving C+ reports, but the reports are successfully approved. #54092
Comments
This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989 |
Triggered auto assignment to @stephanieelliott ( |
Triggered auto assignment to @techievivek ( |
@stephanieelliott, @techievivek Whoops! This issue is 2 days overdue. Let's get this updated quick! |
Hey @techievivek can you take a look at this? |
Sorry, I forgot to post the updates here. I posted a comment in the linked slack thread, this seems to be an issue with authToken expiring when we are fetching the onyx updates for the user. I will bring a discussion in the #eng-chat channel to discuss this further. |
@stephanieelliott, @techievivek Eep! 4 days overdue now. Issues have feelings too... |
Not overdue, will investigate this again today. |
@stephanieelliott @techievivek this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks! |
@stephanieelliott, @techievivek Huh... This is 4 days overdue. Who can take care of this? |
@stephanieelliott, @techievivek 6 days overdue. This is scarier than being forced to listen to Vogon poetry! |
Looking into it now. |
Updates here https://expensify.slack.com/archives/C05LX9D6E07/p1735742835938139?thread_ts=1734003477.036259&cid=C05LX9D6E07, brought an internal discussion here https://expensify.slack.com/archives/C03TQ48KC/p1735746048891849 |
Pushed a PR for adding some logs here https://github.com/Expensify/Web-Expensify/pull/44945, it is a widespread issue when we try to send push notification for IOU action. |
Discussions here https://expensify.slack.com/archives/C03TQ48KC/p1735746048891849 |
We have a clear understanding of the issue now, I am discussing the solution with the team here https://expensify.slack.com/archives/C03TQ48KC/p1736145923335859?thread_ts=1735746048.891849&cid=C03TQ48KC |
@stephanieelliott, @techievivek Whoops! This issue is 2 days overdue. Let's get this updated quick! |
Hey @techievivek have we come up with a solution for this one? |
Will provide an update today. |
@stephanieelliott, @techievivek Still overdue 6 days?! Let's take care of this! |
Pushed a PR for this. |
PR is being actively reviewed |
@stephanieelliott, @techievivek Whoops! This issue is 2 days overdue. Let's get this updated quick! |
@stephanieelliott, @techievivek Huh... This is 4 days overdue. Who can take care of this? |
PR is on staging |
The PR is now live on PROD, and I have confirmed that the Error processing command 'Get' (407 Missing authToken) error log no longer appears in the push notification flow for IOUs or any other actions requiring an authToken. I will go ahead and close this. |
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:
Reproducible in staging?: Needs reproduction
Reproducible in production?: Needs reproduction
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
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: @garrettmknight
Slack conversation (hyperlinked to channel name): quality
Action Performed:
Expected Result:
All the reports got approved
Actual Result:
Unexpected Error’ message displayed for two reports
“Error processing command ‘Get’ (407 Missing authToken), ignoring.”
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Logs
View all open jobs on GitHub
Issue Owner
Current Issue Owner: @techievivekThe text was updated successfully, but these errors were encountered: