Page MenuHomePhabricator

[lib] Fix stack overflow from lastUpdatedTime
ClosedPublic

Authored by ashoat on Dec 13 2024, 9:09 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Mar 20, 5:24 PM
Unknown Object (File)
Tue, Mar 11, 4:12 PM
Unknown Object (File)
Tue, Mar 11, 4:11 PM
Unknown Object (File)
Feb 22 2025, 9:34 PM
Unknown Object (File)
Feb 22 2025, 9:34 PM
Unknown Object (File)
Feb 22 2025, 9:34 PM
Unknown Object (File)
Feb 22 2025, 9:34 PM
Unknown Object (File)
Feb 20 2025, 2:37 PM
Subscribers
None

Details

Summary

This resolves ENG-10016. It looks like when this was changed from an immediately-invoked Promise to a function that returns a Promise, we ended up in a situation where the lastUpdatedTime dereference ended up resolving to the new value rather than the prior value.

Test Plan

Prior to this change, I was able to reproduce the issue in the Linear task. Afterwards, it no longer reproduces

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable