Always reauthenticate when token is unset (#700)

src/components/manifold-auth-token/manifold-auth-token.tsx

@@ -34,9 +34,9 @@ export class ManifoldAuthToken {
   componentWillLoad() {
     this.setExternalToken(this.token);
     if (this.subscribe) {
-      this.unsubscribe = this.subscribe((oldToken?: string, newToken?: string) => {
+      this.unsubscribe = this.subscribe((newToken?: string) => {
         this.internalToken = newToken;
-        if (oldToken && !newToken) {
+        if (!newToken) {
           // changing this to any new string will cause a token refresh. getTime() does that wonderfully.
           this.tick = new Date().getTime().toString();
           this.clear.emit();

src/state/connection.spec.ts

@@ -7,7 +7,7 @@ describe('connection state', () => {
     const subscriber = jest.fn();
     state.subscribe(subscriber);
     state.setAuthToken('new-token');
-    expect(subscriber).toHaveBeenCalledWith('old-token', 'new-token');
+    expect(subscriber).toHaveBeenCalledWith('new-token');
   });
 
   it('will not notify after unsubscribing', () => {

src/state/connection.ts

@@ -4,7 +4,7 @@ import { createRestFetch } from '../utils/restFetch';
 
 const baseWait = 15000;
 
-export type Subscriber = (oldToken?: string, newToken?: string) => void;
+export type Subscriber = (newToken?: string) => void;
 
 const INITIALIZED = 'manifold-connection-initialize';
 
@@ -56,7 +56,7 @@ export class ConnectionState {
   };
 
   setAuthToken = (token: string) => {
-    this.subscribers.forEach(s => s(this.authToken, token));
+    this.subscribers.forEach(s => s(token));
     this.authToken = token;
   };