Page MenuHomePhabricator

[keyserver] Introduce sendMessagesOnAccountCreation
ClosedPublic

Authored by ashoat on May 14 2024, 4:13 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jan 24, 8:18 PM
Unknown Object (File)
Fri, Jan 24, 6:00 AM
Unknown Object (File)
Fri, Jan 24, 6:00 AM
Unknown Object (File)
Fri, Jan 24, 5:59 AM
Unknown Object (File)
Thu, Jan 16, 10:06 AM
Unknown Object (File)
Dec 22 2024, 7:04 PM
Unknown Object (File)
Dec 22 2024, 7:04 PM
Unknown Object (File)
Dec 22 2024, 7:04 PM
Subscribers

Details

Summary

This diff does two things:

  1. Rename processAccountCreationCommon to sendMessagesOnAccountCreation to more accurately reflect its newly reduced scope
  2. Update it to return the list of rawMessageInfos so that it can be used from createAccount (legacy registration endpoint)

I decided to do this while working on ENG-8101, but it doesn't directly address that issue.

Test Plan
  1. I tested account creation via keyserver auth in a multi-keyserver environment
  2. I tested legacy account creation in a legacy environment

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable