Page MenuHomePhabricator

[keyserver] Extend link validation to support thread links
ClosedPublic

Authored by tomek on Tue, Jun 11, 10:00 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Jun 25, 6:33 PM
Unknown Object (File)
Tue, Jun 25, 12:42 AM
Unknown Object (File)
Mon, Jun 24, 2:22 PM
Unknown Object (File)
Sat, Jun 22, 5:20 AM
Unknown Object (File)
Thu, Jun 20, 9:21 PM
Unknown Object (File)
Wed, Jun 19, 1:05 PM
Unknown Object (File)
Wed, Jun 19, 9:38 AM
Unknown Object (File)
Tue, Jun 18, 1:42 PM
Subscribers

Details

Summary

In addition to the previous solution, fetch thread-related data and return the response based on both community and thread data.

https://linear.app/comm/issue/ENG-8249/modify-verify-invite-links-response-to-include-a-thread-id-and-its

Depends on D12354

Test Plan

Checked returned verification status for community and thread links when viewer is and isn't a member.
Also checked if joining a thread link results in joining a community.

These two checks additionally validate that the older client will work correctly with the new solution to links.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable