Page MenuHomePhabricator

[lib] modify queryTimeout
ClosedPublic

Authored by varun on Sep 10 2024, 1:00 PM.
Tags
None
Referenced Files
F3230067: D13277.id.diff
Tue, Nov 12, 6:45 AM
F3229758: D13277.id44017.diff
Tue, Nov 12, 6:00 AM
F3228703: D13277.diff
Tue, Nov 12, 3:13 AM
F3227590: D13277.id44013.diff
Mon, Nov 11, 11:20 PM
Unknown Object (File)
Sat, Nov 9, 9:51 PM
Unknown Object (File)
Oct 11 2024, 10:23 PM
Unknown Object (File)
Oct 11 2024, 10:23 PM
Unknown Object (File)
Oct 11 2024, 10:23 PM
Subscribers

Details

Summary

@ashoat was unable to see usernames on web after logging in following the CSAT launch. he's seen this on native, too. we suspect that the timeout is too short.

more context here: https://linear.app/comm/issue/ENG-9227/ashoat-didnt-see-usernames-on-web-after-logging-in-following-csat

Test Plan

modified identity RPC to sleep for 15 seconds and the previous 10 second timeout was triggered. after changing the timeout duration, I was able to get a response from identity

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ashoat published this revision for review.Sep 10 2024, 1:09 PM
ashoat accepted this revision.
This revision is now accepted and ready to land.Sep 10 2024, 1:10 PM
This revision was automatically updated to reflect the committed changes.