[Glitch] Add source-mapped stacktrace to error message in web UI
Port ff3a11d01d
to glitch-soc
pull/1287/head
parent
c9166a5943
commit
bc4de2f661
|
@ -2,6 +2,7 @@ import React from 'react';
|
||||||
import PropTypes from 'prop-types';
|
import PropTypes from 'prop-types';
|
||||||
import { FormattedMessage } from 'react-intl';
|
import { FormattedMessage } from 'react-intl';
|
||||||
import { preferencesLink } from 'flavours/glitch/util/backend_links';
|
import { preferencesLink } from 'flavours/glitch/util/backend_links';
|
||||||
|
import StackTrace from 'stacktrace-js';
|
||||||
|
|
||||||
export default class ErrorBoundary extends React.PureComponent {
|
export default class ErrorBoundary extends React.PureComponent {
|
||||||
|
|
||||||
|
@ -11,15 +12,29 @@ export default class ErrorBoundary extends React.PureComponent {
|
||||||
|
|
||||||
state = {
|
state = {
|
||||||
hasError: false,
|
hasError: false,
|
||||||
|
errorMessage: undefined,
|
||||||
stackTrace: undefined,
|
stackTrace: undefined,
|
||||||
|
mappedStackTrace: undefined,
|
||||||
componentStack: undefined,
|
componentStack: undefined,
|
||||||
}
|
}
|
||||||
|
|
||||||
componentDidCatch(error, info) {
|
componentDidCatch(error, info) {
|
||||||
this.setState({
|
this.setState({
|
||||||
hasError: true,
|
hasError: true,
|
||||||
|
errorMessage: error.toString(),
|
||||||
stackTrace: error.stack,
|
stackTrace: error.stack,
|
||||||
componentStack: info && info.componentStack,
|
componentStack: info && info.componentStack,
|
||||||
|
mappedStackTrace: undefined,
|
||||||
|
});
|
||||||
|
|
||||||
|
StackTrace.fromError(error).then((stackframes) => {
|
||||||
|
this.setState({
|
||||||
|
mappedStackTrace: stackframes.map((sf) => sf.toString()).join('\n'),
|
||||||
|
});
|
||||||
|
}).catch(() => {
|
||||||
|
this.setState({
|
||||||
|
mappedStackTrace: undefined,
|
||||||
|
});
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -29,13 +44,16 @@ export default class ErrorBoundary extends React.PureComponent {
|
||||||
}
|
}
|
||||||
|
|
||||||
render() {
|
render() {
|
||||||
const { hasError, stackTrace, componentStack } = this.state;
|
const { hasError, errorMessage, stackTrace, mappedStackTrace, componentStack } = this.state;
|
||||||
|
|
||||||
if (!hasError) return this.props.children;
|
if (!hasError) return this.props.children;
|
||||||
|
|
||||||
let debugInfo = '';
|
let debugInfo = '';
|
||||||
if (stackTrace) {
|
if (stackTrace) {
|
||||||
debugInfo += 'Stack trace\n-----------\n\n```\n' + stackTrace.toString() + '\n```';
|
debugInfo += 'Stack trace\n-----------\n\n```\n' + errorMessage + '\n' + stackTrace.toString() + '\n```';
|
||||||
|
}
|
||||||
|
if (mappedStackTrace) {
|
||||||
|
debugInfo += 'Mapped stack trace\n-----------\n\n```\n' + errorMessage + '\n' + mappedStackTrace.toString() + '\n```';
|
||||||
}
|
}
|
||||||
if (componentStack) {
|
if (componentStack) {
|
||||||
if (debugInfo) {
|
if (debugInfo) {
|
||||||
|
|
Loading…
Reference in New Issue