Don't delete channel until it's used

Author: garrettjstevens

packages/apollo-collaboration-server/src/authentication/authentication.service.ts

@@ -186,6 +186,9 @@ export class AuthenticationService {
         username: name,
         role: newUserRole,
       }
+      this.logger.log(
+        `First time login for "${newUser.username}" (${newUser.email})`,
+      )
       user = await this.usersService.addNew(newUser)
     }
     this.logger.debug(`User found in Mongo: ${JSON.stringify(user)}`)
@@ -198,11 +201,7 @@ export class AuthenticationService {
     }
     // Return token with SUCCESS status
     const returnToken = this.jwtService.sign(payload)
-    this.logger.debug(
-      `First time login successful. Apollo token: ${JSON.stringify(
-        returnToken,
-      )}`,
-    )
+    this.logger.debug(`User "${user.username}" has logged in`)
     return { token: returnToken }
   }
 }

packages/jbrowse-plugin-apollo/src/ApolloInternetAccount/model.ts

@@ -107,7 +107,6 @@ const stateModelFactory = (configSchema: ApolloInternetAccountConfigModel) => {
           if (
             event.data.name !== `JBrowseAuthWindow-${self.internetAccountId}`
           ) {
-            this.deleteMessageChannel()
             return
           }
           const redirectUriWithInfo = event.data.redirectUri

packages/jbrowse-plugin-apollo/src/session/session.ts

@@ -345,6 +345,10 @@ export function extendSession(
                   console.error(error)
                   continue
                 }
+                // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+                if (!jbrowseConfig.configuration.ApolloPlugin.hasRole) {
+                  continue
+                }
                 // eslint-disable-next-line @typescript-eslint/no-unsafe-call
                 reloadPluginManagerCallback(
                   jbrowseConfig,