fix: remove unnecessary "await" in baseUserRepository.js

Fixes a small bug where we had an `await` call on a function
that doesn't return a `Promise`.

Technically this is fine, because Node will basically
"promisify" the value, but it isn't necessary and creates
some extra work for the runtime. Specifically, in this
situation, Node will wrap the value in a Promise implicitly
and pause execution of the surrounding `async` function to
queue up that Promise for resolution.

So removing it is semantically equivalent, but improves
runtime behavior by eliminating this extra work.

Signed-off-by: Andrew Lilley Brinker <abrinker@mitre.org>

Author: alilleybrinker

src/repositories/baseUserRepository.js

@@ -293,7 +293,7 @@ class BaseUserRepository extends BaseRepository {
   async isAdminOrSecretariat (orgShortName, username, requesterOrg, options = {}, isRegistryObject = true) {
     const baseOrgRepository = new BaseOrgRepository()
     const org = await baseOrgRepository.findOneByShortName(requesterOrg)
-    if (await baseOrgRepository.isSecretariat(org) || await this.isAdmin(username, orgShortName, options, isRegistryObject)) {
+    if (baseOrgRepository.isSecretariat(org) || await this.isAdmin(username, orgShortName, options, isRegistryObject)) {
       return true
     }
     return false