[lib] set `dataIsBackedUp` to be false in `keyserverThreadProtocol`
ClosedPublic
Actions

Authored by kamil on Jun 17 2025, 12:58 PM.

Details

Reviewers

tomek
bartek

Commits

rCOMM6e66b7e3a802: [lib] set `dataIsBackedUp` to be false in `keyserverThreadProtocol`

Summary

This is an oversight from D14820, it was hard to discover it because I wasn't able to test the actual impact on backup then. This is still ignored on the database layer (still in progress), so there is no regression on master.

Depends on D14820

Test Plan

N/A

Diff Detail

Repository

rCOMM Comm

Branch

publish-2

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

kamil created this revision.Jun 17 2025, 12:58 PM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptJun 17 2025, 12:58 PM

kamil published this revision for review.Jun 17 2025, 12:58 PM

kamil edited the summary of this revision. (Show Details)

Harbormaster failed remote builds in B34432: Diff 48525!Jun 17 2025, 1:11 PM

Harbormaster completed remote builds in B34432: Diff 48525.Jun 17 2025, 1:34 PM

tomek accepted this revision.Jun 23 2025, 11:55 AM

This revision is now accepted and ready to land.Jun 23 2025, 11:55 AM

Closed by commit rCOMM6e66b7e3a802: [lib] set `dataIsBackedUp` to be false in `keyserverThreadProtocol`. · Explain WhyJun 23 2025, 3:04 PM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM6e66b7e3a802: [lib] set `dataIsBackedUp` to be false in `keyserverThreadProtocol`.

Revision Contents
Changeset List

Path

Size

lib/

shared/

threads/

protocols/

keyserver-thread-protocol.js

2 lines

Diff 48525

View Options

lib/shared/threads/protocols/keyserver-thread-protocol.js

[lib] set `dataIsBackedUp` to be false in `keyserverThreadProtocol`ClosedPublicActions