From d143ba95000f0b03685583fb9442863b9270e5a9 Mon Sep 17 00:00:00 2001 From: Florian Dold Date: Mon, 13 Feb 2017 03:28:36 +0100 Subject: [PATCH] check for contract freshness --- src/content_scripts/notify.ts | 10 ++++++++-- src/types.ts | 3 +++ 2 files changed, 11 insertions(+), 2 deletions(-) diff --git a/src/content_scripts/notify.ts b/src/content_scripts/notify.ts index 10b988c43..d29e5913d 100644 --- a/src/content_scripts/notify.ts +++ b/src/content_scripts/notify.ts @@ -173,7 +173,9 @@ namespace TalerNotify { (detail: any, sendResponse: (msg: any) => void): void; } - function downloadContract(url: string): Promise { + function downloadContract(url: string, nonce: string): Promise { + let parsed_url = URI(url); + url = parsed_url.setQuery({nonce}).href(); // FIXME: include and check nonce! return new Promise((resolve, reject) => { const contract_request = new XMLHttpRequest(); @@ -356,7 +358,11 @@ namespace TalerNotify { return; } if (msg.contract_url) { - let proposal = await downloadContract(msg.contract_url); + let nonce = Math.round(Math.random() * 0xFFFF).toString() + let proposal = await downloadContract(msg.contract_url, nonce); + if (proposal.data.nonce != nonce) { + console.error("stale contract"); + } await processProposal(proposal); return; } diff --git a/src/types.ts b/src/types.ts index ab50c7369..30e9b7856 100644 --- a/src/types.ts +++ b/src/types.ts @@ -444,6 +444,9 @@ export class Contract { @Checkable.Optional(Checkable.String) summary?: string; + @Checkable.Optional(Checkable.String) + nonce?: string; + @Checkable.Value(AmountJson) amount: AmountJson;