Show error message when attempting to connect to a websocket

This commit is contained in:
Dane Everitt 2020-04-17 12:41:47 -07:00
parent 41a94c66e4
commit 5252f3f9d3
No known key found for this signature in database
GPG key ID: EEA66103B3D71F53
3 changed files with 57 additions and 16 deletions

View file

@ -154,7 +154,7 @@ export default () => {
</div> </div>
</Can> </Can>
</div> </div>
<div className={'flex-1 mx-4 mr-4'}> <div className={'flex-1 ml-4'}>
<SuspenseSpinner> <SuspenseSpinner>
<ChunkedConsole/> <ChunkedConsole/>
<ChunkedStatGraphs/> <ChunkedStatGraphs/>

View file

@ -1,11 +1,15 @@
import React, { useEffect } from 'react'; import React, { useEffect, useState } from 'react';
import { Websocket } from '@/plugins/Websocket'; import { Websocket } from '@/plugins/Websocket';
import { ServerContext } from '@/state/server'; import { ServerContext } from '@/state/server';
import getWebsocketToken from '@/api/server/getWebsocketToken'; import getWebsocketToken from '@/api/server/getWebsocketToken';
import ContentContainer from '@/components/elements/ContentContainer';
import { CSSTransition } from 'react-transition-group';
import Spinner from '@/components/elements/Spinner';
export default () => { export default () => {
const server = ServerContext.useStoreState(state => state.server.data); const server = ServerContext.useStoreState(state => state.server.data);
const { instance } = ServerContext.useStoreState(state => state.socket); const [ error, setError ] = useState(false);
const { connected, instance } = ServerContext.useStoreState(state => state.socket);
const setServerStatus = ServerContext.useStoreActions(actions => actions.status.setServerStatus); const setServerStatus = ServerContext.useStoreActions(actions => actions.status.setServerStatus);
const { setInstance, setConnectionState } = ServerContext.useStoreActions(actions => actions.socket); const { setInstance, setConnectionState } = ServerContext.useStoreActions(actions => actions.socket);
@ -15,6 +19,16 @@ export default () => {
.catch(error => console.error(error)); .catch(error => console.error(error));
}; };
useEffect(() => {
connected && setError(false);
}, [ connected ]);
useEffect(() => {
return () => {
instance && instance.close();
};
}, [ instance ]);
useEffect(() => { useEffect(() => {
// If there is already an instance or there is no server, just exit out of this process // If there is already an instance or there is no server, just exit out of this process
// since we don't need to make a new connection. // since we don't need to make a new connection.
@ -26,7 +40,10 @@ export default () => {
socket.on('auth success', () => setConnectionState(true)); socket.on('auth success', () => setConnectionState(true));
socket.on('SOCKET_CLOSE', () => setConnectionState(false)); socket.on('SOCKET_CLOSE', () => setConnectionState(false));
socket.on('SOCKET_ERROR', () => setConnectionState(false)); socket.on('SOCKET_ERROR', () => {
setError(true);
setConnectionState(false);
});
socket.on('status', (status) => setServerStatus(status)); socket.on('status', (status) => setServerStatus(status));
socket.on('daemon error', message => { socket.on('daemon error', message => {
@ -47,5 +64,19 @@ export default () => {
.catch(error => console.error(error)); .catch(error => console.error(error));
}, [ server ]); }, [ server ]);
return null; return (
error ?
<CSSTransition timeout={250} in={true} appear={true} classNames={'fade'}>
<div className={'bg-red-500 py-2'}>
<ContentContainer className={'flex items-center justify-center'}>
<Spinner size={'tiny'}/>
<p className={'ml-2 text-sm text-red-100'}>
We're having some trouble connecting to the console, please wait...
</p>
</ContentContainer>
</div>
</CSSTransition>
:
null
);
}; };

View file

@ -9,6 +9,12 @@ export const SOCKET_EVENTS = [
]; ];
export class Websocket extends EventEmitter { export class Websocket extends EventEmitter {
// Timer instance for this socket.
private timer: any = null;
// The backoff for the timer, in milliseconds.
private backoff = 5000;
// The socket instance being tracked. // The socket instance being tracked.
private socket: Sockette | null = null; private socket: Sockette | null = null;
@ -25,6 +31,7 @@ export class Websocket extends EventEmitter {
// Connects to the websocket instance and sets the token for the initial request. // Connects to the websocket instance and sets the token for the initial request.
connect (url: string): this { connect (url: string): this {
this.url = url; this.url = url;
this.socket = new Sockette(`${this.url}`, { this.socket = new Sockette(`${this.url}`, {
onmessage: e => { onmessage: e => {
try { try {
@ -35,6 +42,10 @@ export class Websocket extends EventEmitter {
} }
}, },
onopen: () => { onopen: () => {
// Clear the timers, we managed to connect just fine.
this.timer && clearTimeout(this.timer);
this.backoff = 5000;
this.emit('SOCKET_OPEN'); this.emit('SOCKET_OPEN');
this.authenticate(); this.authenticate();
}, },
@ -43,15 +54,19 @@ export class Websocket extends EventEmitter {
this.authenticate(); this.authenticate();
}, },
onclose: () => this.emit('SOCKET_CLOSE'), onclose: () => this.emit('SOCKET_CLOSE'),
onerror: () => this.emit('SOCKET_ERROR'), onerror: error => this.emit('SOCKET_ERROR', error),
}); });
return this; this.timer = setTimeout(() => {
} this.backoff = (this.backoff + 2500 >= 20000) ? 20000 : this.backoff + 2500;
this.socket && this.socket.close();
clearTimeout(this.timer);
// Returns the URL connected to for the socket. // Re-attempt connecting to the socket.
getSocketUrl (): string | null { this.connect(url);
return this.url; }, this.backoff);
return this;
} }
// Sets the authentication token to use when sending commands back and forth // Sets the authentication token to use when sending commands back and forth
@ -66,11 +81,6 @@ export class Websocket extends EventEmitter {
return this; return this;
} }
// Returns the token being used at the current moment.
getToken (): string {
return this.token;
}
authenticate () { authenticate () {
if (this.url && this.token) { if (this.url && this.token) {
this.send('auth', this.token); this.send('auth', this.token);