Properly detect user signing cancellation on Metamask, Parity signer and Ledger
This commit is contained in:
		@@ -108,7 +108,7 @@ export class EthWethConversionButton extends React.Component<
 | 
			
		||||
            const errMsg = `${err}`;
 | 
			
		||||
            if (_.includes(errMsg, BlockchainCallErrs.UserHasNoAssociatedAddresses)) {
 | 
			
		||||
                this.props.dispatcher.updateShouldBlockchainErrDialogBeOpen(true);
 | 
			
		||||
            } else if (!_.includes(errMsg, 'User denied transaction')) {
 | 
			
		||||
            } else if (!utils.didUserDenyWeb3Request(errMsg)) {
 | 
			
		||||
                utils.consoleLog(`Unexpected error encountered: ${err}`);
 | 
			
		||||
                utils.consoleLog(err.stack);
 | 
			
		||||
                const errorMsg =
 | 
			
		||||
 
 | 
			
		||||
@@ -573,7 +573,7 @@ export class FillOrder extends React.Component<FillOrderProps, FillOrderState> {
 | 
			
		||||
                isFilling: false,
 | 
			
		||||
            });
 | 
			
		||||
            const errMsg = `${err}`;
 | 
			
		||||
            if (_.includes(errMsg, 'User denied transaction signature')) {
 | 
			
		||||
            if (utils.didUserDenyWeb3Request(errMsg)) {
 | 
			
		||||
                return;
 | 
			
		||||
            }
 | 
			
		||||
            globalErrMsg = 'Failed to fill order, please refresh and try again';
 | 
			
		||||
@@ -653,7 +653,7 @@ export class FillOrder extends React.Component<FillOrderProps, FillOrderState> {
 | 
			
		||||
                isCancelling: false,
 | 
			
		||||
            });
 | 
			
		||||
            const errMsg = `${err}`;
 | 
			
		||||
            if (_.includes(errMsg, 'User denied transaction signature')) {
 | 
			
		||||
            if (utils.didUserDenyWeb3Request(errMsg)) {
 | 
			
		||||
                return;
 | 
			
		||||
            }
 | 
			
		||||
            globalErrMsg = 'Failed to cancel order, please refresh and try again';
 | 
			
		||||
 
 | 
			
		||||
@@ -78,7 +78,7 @@ export class AllowanceToggle extends React.Component<AllowanceToggleProps, Allow
 | 
			
		||||
                isSpinnerVisible: false,
 | 
			
		||||
            });
 | 
			
		||||
            const errMsg = `${err}`;
 | 
			
		||||
            if (_.includes(errMsg, 'User denied transaction')) {
 | 
			
		||||
            if (utils.didUserDenyWeb3Request(errMsg)) {
 | 
			
		||||
                return;
 | 
			
		||||
            }
 | 
			
		||||
            utils.consoleLog(`Unexpected error encountered: ${err}`);
 | 
			
		||||
 
 | 
			
		||||
@@ -73,7 +73,7 @@ export class SendButton extends React.Component<SendButtonProps, SendButtonState
 | 
			
		||||
            if (_.includes(errMsg, BlockchainCallErrs.UserHasNoAssociatedAddresses)) {
 | 
			
		||||
                this.props.dispatcher.updateShouldBlockchainErrDialogBeOpen(true);
 | 
			
		||||
                return;
 | 
			
		||||
            } else if (!_.includes(errMsg, 'User denied transaction')) {
 | 
			
		||||
            } else if (!utils.didUserDenyWeb3Request(errMsg)) {
 | 
			
		||||
                utils.consoleLog(`Unexpected error encountered: ${err}`);
 | 
			
		||||
                utils.consoleLog(err.stack);
 | 
			
		||||
                this.props.onError();
 | 
			
		||||
 
 | 
			
		||||
@@ -519,7 +519,7 @@ export class TokenBalances extends React.Component<TokenBalancesProps, TokenBala
 | 
			
		||||
                this.props.dispatcher.updateShouldBlockchainErrDialogBeOpen(true);
 | 
			
		||||
                return false;
 | 
			
		||||
            }
 | 
			
		||||
            if (_.includes(errMsg, 'User denied transaction')) {
 | 
			
		||||
            if (utils.didUserDenyWeb3Request(errMsg)) {
 | 
			
		||||
                return false;
 | 
			
		||||
            }
 | 
			
		||||
            utils.consoleLog(`Unexpected error encountered: ${err}`);
 | 
			
		||||
 
 | 
			
		||||
@@ -183,7 +183,7 @@ export const utils = {
 | 
			
		||||
    // after a user was prompted to sign a message or send a transaction and decided to
 | 
			
		||||
    // reject the request.
 | 
			
		||||
    didUserDenyWeb3Request(errMsg: string) {
 | 
			
		||||
        const metamaskDenialErrMsg = 'User denied message';
 | 
			
		||||
        const metamaskDenialErrMsg = 'User denied';
 | 
			
		||||
        const paritySignerDenialErrMsg = 'Request has been rejected';
 | 
			
		||||
        const ledgerDenialErrMsg = 'Invalid status 6985';
 | 
			
		||||
        const isUserDeniedErrMsg =
 | 
			
		||||
 
 | 
			
		||||
		Reference in New Issue
	
	Block a user