fix: don't repeatedly use invalid SRC_ACCESS_TOKEN env variable

Author: olafurpg

src/settings/accessTokenSetting.ts

@@ -6,6 +6,7 @@ import { endpointSetting } from './endpointSetting'
 import { readConfiguration } from './readConfiguration'
 
 let cachedAccessToken: Promise<string> | undefined
+const invalidAccessTokens = new Set<string>()
 
 export function accessTokenSetting(): Promise<string> {
     const fromSettings = readConfiguration().get<string>('accessToken', '')
@@ -14,14 +15,15 @@ export function accessTokenSetting(): Promise<string> {
     }
 
     const environmentVariable = process.env.SRC_ACCESS_TOKEN
-    if (environmentVariable) {
+    if (environmentVariable && !invalidAccessTokens.has(environmentVariable)) {
         return Promise.resolve(environmentVariable)
     }
 
     return promptUserForAccessTokenSetting()
 }
 
 export async function deleteAccessTokenSetting(tokenValueToDelete: string): Promise<void> {
+    invalidAccessTokens.add(tokenValueToDelete)
     const currentValue = readConfiguration().get<string>('accessToken')
     if (currentValue === tokenValueToDelete) {
         cachedAccessToken = undefined