A channel's newest posts miss texts that arrived after its status was cached #296
Labels
No labels
area/ci
area/desktop
area/packaging
area/protocol
area/reading
area/ui
area/web
area/writing
kind/bug
kind/feature
kind/infra
kind/request
needs/user
parked
perf
security
upstream
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
littlefox/neokom#296
Loading…
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Found while building #282.
Kom::channel_page(c, None, n)starts from the cachedhighest_local_noofc(uconfs), and new-text messages didn't update that cache. So a channel's newest page, and the thread list's new threads, missed texts written after the conference status was first fetched, until the cache entry was evicted.Fix (in #282's PR):
on_asyncraises the cachedhighest_local_noof each recipient conference to the new text's local number. Covered by thechannel_threadsintegration test (a late reply is listed).Fixed in #297 (on_async NewText raises each recipient's cached highest_local_no). Covered by the devserver channel_threads test, which failed before the fix. Main run #485 green.