Page MenuHomePhabricator

[keyserver] Use keyserverID instead of userID when creating invite link blobs
ClosedPublic

Authored by tomek on Mon, Apr 22, 4:49 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, May 12, 7:45 AM
Unknown Object (File)
Sat, May 11, 11:53 PM
Unknown Object (File)
Sat, May 11, 8:22 PM
Unknown Object (File)
Fri, May 10, 12:45 AM
Unknown Object (File)
Mon, May 6, 8:07 PM
Unknown Object (File)
Sat, May 4, 11:56 AM
Unknown Object (File)
Sat, May 4, 9:41 AM
Unknown Object (File)
Thu, May 2, 10:56 AM
Subscribers

Details

Summary

For each link we create a blob that contains a keyserver ID that is later used as a part of the override on the client side. In a single-keyserver world, this value should be equal to 256 because this is the ID a client uses as a keyserver ID. Instead of using the correct value, we were using the ID of a user, taken from Identity - so it was a multi-keyserver keyserver ID.

This issue wasn't affecting prod because the ID of the keyserver on prod Identity is equal to 256.

https://linear.app/comm/issue/ENG-7789/invite-links-in-single-keyserver-world-fail

Test Plan

Tested if creating a new link and accepting it works with both single- and multi-keyserver IDs. (testing multi-keyserver ID was performed by having a client with usingCommservicesAccessToken set to false, and a keyserver with usingIdentityCredentials set to true).

Diff Detail

Repository
rCOMM Comm
Branch
master
Lint
No Lint Coverage
Unit
No Test Coverage