NodeBB / NodeBB / 22866930047

25%

Build Type

push

github

Committed by barisusakli

Commit Message

fix: promises in groups.leave

speed up user.delete

user.delete calls `groups.leaveAllGroups` which calls rejectMembership with 500+ groups. This function then tries to remove the user from `group:<group>:pending` and `group:<group>:invited` sets so a total for 1k sets. You can't be invited or request membership to privilege groups so filter the groups before sending to rejectMembership

clearGroupTitleIfSet function tries to remove the group title from the user. It was only skipping privilege groups and registered-users, but unverified-users & verified users weren't added to the check

Messaging.leaveRooms, make a single call to isUserInRoom and passing an array of roomIds

In user.delete, check utils.isNumber(uid) once.

Call deleteVotes/deleteChats/revokeAllSessions in Promise.all

If user is local dont call activitypub.actors.remove(), this saves a db call to `await db.isSortedSetMember('usersRemote:lastCrawled', id);`

Coverage Stats

13360 of 18277 branches covered (73.1%)

19 of 19 new or added lines in 3 files covered. (100.0%)

1 existing line in 1 file now uncovered.

28228 of 33002 relevant lines covered (85.53%)

3323.59 hits per line