Page MenuHomePhabricator

[keyserver][lib] Block users from creating reserved invite links
ClosedPublic

Authored by tomek on Jul 25 2023, 3:39 AM.
Tags
None
Referenced Files
F3526337: D8615.id29011.diff
Mon, Dec 23, 9:50 PM
F3524915: D8615.diff
Mon, Dec 23, 3:07 PM
Unknown Object (File)
Fri, Dec 13, 10:54 PM
Unknown Object (File)
Sat, Dec 7, 4:37 AM
Unknown Object (File)
Nov 23 2024, 6:55 PM
Unknown Object (File)
Nov 23 2024, 4:19 PM
Unknown Object (File)
Nov 23 2024, 2:36 PM
Unknown Object (File)
Nov 20 2024, 5:55 PM
Subscribers
None

Details

Summary

A user should not be able to create a link which is included in a list of reserved usernames. The copy is similar to a message that is shown when a user tries to create an account with reserved username: /native/account/registration/registration-server-call.js, but I'm not sure if we should mention support email (the user is already registered and can contact us on Comm).

Depends on D8527

Test Plan

Try to create /comm invite link - a message should be displayed.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

tomek requested review of this revision.Jul 25 2023, 4:27 AM
This revision is now accepted and ready to land.Jul 26 2023, 1:36 AM
lib/shared/invite-links.js
8–9 ↗(On Diff #29011)

It's a little inconsistent how we're using full sentences here, but not in the other ones. Should we add a period to the other ones?

lib/shared/invite-links.js
8–9 ↗(On Diff #29011)

It is probably a good idea - going to update the original diff