Page MenuHomePhabricator
Files F3356633

D12392.id41211.diff
No OneTemporary
Actions

D12392.id41211.diff
View Options

diff --git a/keyserver/src/fetchers/link-fetchers.js b/keyserver/src/fetchers/link-fetchers.js
--- a/keyserver/src/fetchers/link-fetchers.js
+++ b/keyserver/src/fetchers/link-fetchers.js
@@ -1,5 +1,9 @@
// @flow
+import {
+ FUTURE_CODE_VERSION,
+ hasMinCodeVersion,
+} from 'lib/shared/version-utils.js';
import type {
InviteLinkWithHolder,
InviteLinkVerificationRequest,
@@ -15,14 +19,21 @@
request: InviteLinkVerificationRequest,
): Promise<InviteLinkVerificationResponse> {
const query = SQL`
- SELECT c.name, i.community AS communityID, m.role
+ SELECT c.name AS communityName, c.id AS communityID,
+ cm.role AS communityRole, t.name AS threadName, t.id AS threadID,
+ tm.role AS threadRole
FROM invite_links i
INNER JOIN threads c ON c.id = i.community
- LEFT JOIN memberships m
- ON m.thread = i.community
- AND m.user = ${viewer.loggedIn ? viewer.userID : null}
+ LEFT JOIN memberships cm
+ ON cm.thread = c.id
+ AND cm.user = ${viewer.loggedIn ? viewer.userID : null}
+ LEFT JOIN threads t ON t.id = i.thread
+ LEFT JOIN memberships tm
+ ON tm.thread = t.id
+ AND tm.user = ${viewer.loggedIn ? viewer.userID : null}
WHERE i.name = ${request.secret}
AND c.community IS NULL
+ AND (t.community = c.id OR i.thread IS NULL)
`;
const [result] = await dbQuery(query);
if (result.length === 0) {
@@ -31,15 +42,38 @@
};
}
- const { name, communityID, role } = result[0];
- const status = role > 0 ? 'already_joined' : 'valid';
- return {
- status,
+ const supportsThreadLinks = hasMinCodeVersion(viewer.platformDetails, {
+ native: FUTURE_CODE_VERSION,
+ web: FUTURE_CODE_VERSION,
+ });
+ const {
+ communityName,
+ communityID,
+ communityRole,
+ threadName,
+ threadID,
+ threadRole,
+ } = result[0];
+ const communityStatus = communityRole > 0 ? 'already_joined' : 'valid';
+ const communityResult = {
+ status: communityStatus,
community: {
- name,
+ name: communityName,
id: communityID.toString(),
},
};
+ if (!threadID || !supportsThreadLinks) {
+ return communityResult;
+ }
+ const threadStatus = threadRole > 0 ? 'already_joined' : 'valid';
+ return {
+ ...communityResult,
+ status: threadStatus,
+ thread: {
+ name: threadName,
+ id: threadID.toString(),
+ },
+ };
}
async function checkIfInviteLinkIsValid(
diff --git a/lib/types/link-types.js b/lib/types/link-types.js
--- a/lib/types/link-types.js
+++ b/lib/types/link-types.js
@@ -15,6 +15,10 @@
+name: string,
+id: string,
},
+ +thread?: {
+ +name: string,
+ +id: string,
+ },
}
| {
+status: 'invalid' | 'expired',
diff --git a/lib/types/validators/link-validators.js b/lib/types/validators/link-validators.js
--- a/lib/types/validators/link-validators.js
+++ b/lib/types/validators/link-validators.js
@@ -14,14 +14,20 @@
links: t.list(inviteLinkValidator),
});
+const threadDataValidator = tShape<{
+ +name: string,
+ +id: string,
+}>({
+ name: t.String,
+ id: tID,
+});
+
export const inviteLinkVerificationResponseValidator: TUnion<InviteLinkVerificationResponse> =
t.union([
tShape({
status: t.enums.of(['valid', 'already_joined']),
- community: tShape({
- name: t.String,
- id: tID,
- }),
+ community: threadDataValidator,
+ thread: t.maybe(threadDataValidator),
}),
tShape({
status: t.enums.of(['invalid', 'expired']),

File Metadata

Mime Type
text/plain
Expires
Sun, Nov 24, 7:43 PM (8 h, 35 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
2577155
Default Alt Text
D12392.id41211.diff (3 KB)

Event Timeline