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
F2773164: D8615.id29177.diff
Fri, Sep 20, 1:55 AM
F2773163: D8615.id29173.diff
Fri, Sep 20, 1:55 AM
F2773161: D8615.id29011.diff
Fri, Sep 20, 1:55 AM
Unknown Object (File)
Sun, Sep 15, 4:13 PM
Unknown Object (File)
Mon, Sep 9, 12:07 AM
Unknown Object (File)
Fri, Sep 6, 10:05 PM
Unknown Object (File)
Mon, Sep 2, 3:51 PM
Unknown Object (File)
Aug 15 2024, 12:42 AM
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