ensure csr is saved when updating username

TryQuiet · Oct 25, 2024 · 0cad2d4 · 0cad2d4
1 parent a80ab2e
commit 0cad2d4
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 9 deletions.
diff --git a/packages/backend/src/nest/connections-manager/connections-manager.service.ts b/packages/backend/src/nest/connections-manager/connections-manager.service.ts
@@ -460,7 +460,7 @@ export class ConnectionsManagerService extends EventEmitter implements OnModuleI
       userCsr = await createUserCsr(createUserCsrPayload)
     } catch (e) {
       emitError(this.serverIoProvider.io, {
-        type: SocketActionTypes.ADD_CSR,
+        type: SocketActionTypes.CREATE_USER_CSR,
         message: ErrorMessages.USER_CSR_CREATION_FAILED,
         community: communityId,
       })
@@ -470,6 +470,9 @@ export class ConnectionsManagerService extends EventEmitter implements OnModuleI
     identity = { ...identity, userCsr: userCsr, nickname: nickname }
     this.logger.info('Created user CSR')
     await this.storageService.setIdentity(identity)
+    if (payload.isUsernameTaken) {
+      await this.storageService.saveCSR({ csr: userCsr.userCsr })
+    }
     return identity
   }
 

diff --git a/packages/state-manager/src/sagas/identity/registerUsername/registerUsername.saga.ts b/packages/state-manager/src/sagas/identity/registerUsername/registerUsername.saga.ts
@@ -32,7 +32,7 @@ export function* registerUsernameSaga(
   logger.info(`Found community ${community?.id} has CA?: ${community?.CA !== null}`)
 
   logger.info('Emitting CREATE_USER_CSR')
-  const payload: InitUserCsrPayload = { communityId: community.id, nickname: nickname }
+  const payload: InitUserCsrPayload = { communityId: community.id, nickname, isUsernameTaken }
   const identity: Identity = yield* apply(
     socket,
     socket.emitWithAck,
@@ -43,12 +43,8 @@ export function* registerUsernameSaga(
 
   if (community.CA?.rootCertString) {
     yield* put(communitiesActions.createCommunity(community.id))
-  } else {
-    if (!isUsernameTaken) {
-      logger.info('Username is not taken, launching community')
-      yield* put(communitiesActions.launchCommunity(community.id))
-    } else {
-      yield* put(identityActions.saveUserCsr())
-    }
+  } else if (!isUsernameTaken) {
+    logger.info('Username is not taken, launching community')
+    yield* put(communitiesActions.launchCommunity(community.id))
   }
 }
diff --git a/packages/types/src/identity.ts b/packages/types/src/identity.ts
@@ -47,6 +47,7 @@ export interface IdentityUpdatePayload {
 export interface InitUserCsrPayload {
   communityId: string
   nickname: string
+  isUsernameTaken?: boolean
 }
 
 export interface CreateUserCsrPayload {