Page MenuHomePhabricator
Differential D11496

[lib] Fix keyserver sending updated RoleInfos to old clients
ClosedPublic
Actions

Authored by ashoat on Mar 31 2024, 7:14 PM.
Tags
None
Referenced Files
F3360474: D11496.id38643.diff
Sun, Nov 24, 12:47 PM
F3360156: D11496.id38609.diff
Sun, Nov 24, 12:20 PM
F3359803: D11496.diff
Sun, Nov 24, 10:58 AM
Unknown Object (File)
Wed, Nov 13, 12:59 AM
Unknown Object (File)
Wed, Nov 13, 12:04 AM
Unknown Object (File)
Wed, Nov 13, 12:04 AM
Unknown Object (File)
Tue, Nov 12, 10:08 PM
Unknown Object (File)
Oct 11 2024, 10:12 AM
View All 47 Files
Subscribers
atul
tomek

Details

Reviewers
atul
Commits
rCOMMfe3428384c93: [lib] Fix keyserver sending updated RoleInfos to old clients
Summary

@atul's recent changes caused a major issue where new RoleInfos were being delivered to older clients, causing validation failures.

This diff makes it so keyserver can continue outputting the old-style RoleInfos to current clients.

Test Plan

I deployed this to prod and:

  1. Confirmed web app refreshed without issue
  2. Confirmed web app could log out / back in
  3. Confirmed native could receive a message without seeing validation errors
  4. Confirmed native could press a notif without seeing validation errors
  5. Confirmed native could log out / back in

Diff Detail

Repository
rCOMM Comm
Branch
ashoat/fixprod
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

ashoat created this revision.Mar 31 2024, 7:14 PM
Herald added a subscriber: tomek. · View Herald TranscriptMar 31 2024, 7:14 PM
ashoat published this revision for review.Mar 31 2024, 7:15 PM
Harbormaster completed remote builds in B27876: Diff 38609.Mar 31 2024, 7:30 PM
atul accepted this revision.Apr 1 2024, 12:43 PM

Thanks for catching and resolving this.

I definitely thought through this scenario, specifically when I introduced https://phab.comm.dev/D11286... shouldn't have missed it.

This revision is now accepted and ready to land.Apr 1 2024, 12:43 PM
Closed by commit rCOMMfe3428384c93: [lib] Fix keyserver sending updated RoleInfos to old clients (authored by ashoat). · Explain WhyApr 1 2024, 5:55 PM
This revision was automatically updated to reflect the committed changes.
ashoat added a commit: rCOMMfe3428384c93: [lib] Fix keyserver sending updated RoleInfos to old clients.

Revision Contents
Changeset List

Diff 38609

View Options

lib/permissions/minimally-encoded-raw-thread-info-validators.js

Loading...
View Options

lib/shared/thread-utils.js

Loading...
View Options

lib/types/minimally-encoded-thread-permissions-types.js

Loading...