complain about merchant base URL mismatch, fixing #6616
This commit is contained in:
parent
ce326361b4
commit
0d37ec5e91
@ -31,6 +31,14 @@ import { TalerErrorCode } from "../TalerErrorCode";
|
||||
* but the error has already been reported by writing it to the database.
|
||||
*/
|
||||
export class OperationFailedAndReportedError extends Error {
|
||||
static fromCode(
|
||||
ec: TalerErrorCode,
|
||||
message: string,
|
||||
details: Record<string, unknown>,
|
||||
): OperationFailedAndReportedError {
|
||||
return new OperationFailedAndReportedError(makeErrorDetails(ec, message, details));
|
||||
}
|
||||
|
||||
constructor(public operationError: TalerErrorDetails) {
|
||||
super(operationError.message);
|
||||
|
||||
|
@ -56,7 +56,11 @@ import * as Amounts from "../util/amounts";
|
||||
import { AmountJson } from "../util/amounts";
|
||||
import { Logger } from "../util/logging";
|
||||
import { parsePayUri } from "../util/taleruri";
|
||||
import { guardOperationException, OperationFailedError } from "./errors";
|
||||
import {
|
||||
guardOperationException,
|
||||
OperationFailedAndReportedError,
|
||||
OperationFailedError,
|
||||
} from "./errors";
|
||||
import { createRefreshGroup, getTotalRefreshCost } from "./refresh";
|
||||
import { InternalWalletState, EXCHANGE_COINS_LOCK } from "./state";
|
||||
import {
|
||||
@ -661,6 +665,20 @@ async function processDownloadProposalImpl(
|
||||
);
|
||||
const fulfillmentUrl = parsedContractTerms.fulfillment_url;
|
||||
|
||||
const baseUrlForDownload = proposal.merchantBaseUrl;
|
||||
const baseUrlFromContractTerms = parsedContractTerms.merchant_base_url;
|
||||
|
||||
if (baseUrlForDownload !== baseUrlFromContractTerms) {
|
||||
throw OperationFailedAndReportedError.fromCode(
|
||||
TalerErrorCode.WALLET_CONTRACT_TERMS_BASE_URL_MISMATCH,
|
||||
"merchant base URL mismatch",
|
||||
{
|
||||
baseUrlForDownload,
|
||||
baseUrlFromContractTerms,
|
||||
},
|
||||
);
|
||||
}
|
||||
|
||||
await ws.db.runWithWriteTransaction(
|
||||
[Stores.proposals, Stores.purchases],
|
||||
async (tx) => {
|
||||
|
Loading…
Reference in New Issue
Block a user