Skip to content

Commit 4df3cd6

Browse files
committed
fix: delete current user async so we do not spend too much time on disconnection
1 parent ffd8b52 commit 4df3cd6

File tree

1 file changed

+10
-11
lines changed

1 file changed

+10
-11
lines changed

src/session.ts

Lines changed: 10 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -69,25 +69,24 @@ export class Session extends EventEmitter<{
6969
this.emit("disconnect");
7070
return;
7171
}
72-
try {
73-
await this.apiClient.deleteDatabaseUser({
72+
void this.apiClient
73+
.deleteDatabaseUser({
7474
params: {
7575
path: {
7676
groupId: this.connectedAtlasCluster.projectId,
7777
username: this.connectedAtlasCluster.username,
7878
databaseName: "admin",
7979
},
8080
},
81+
})
82+
.catch((err: unknown) => {
83+
const error = err instanceof Error ? err : new Error(String(err));
84+
logger.error(
85+
LogId.atlasDeleteDatabaseUserFailure,
86+
"atlas-connect-cluster",
87+
`Error deleting previous database user: ${error.message}`
88+
);
8189
});
82-
} catch (err: unknown) {
83-
const error = err instanceof Error ? err : new Error(String(err));
84-
85-
logger.error(
86-
LogId.atlasDeleteDatabaseUserFailure,
87-
"atlas-connect-cluster",
88-
`Error deleting previous database user: ${error.message}`
89-
);
90-
}
9190
this.connectedAtlasCluster = undefined;
9291

9392
this.emit("disconnect");

0 commit comments

Comments
 (0)