From 92ba02a8d593153c7dd14815857075257b23cd1f Mon Sep 17 00:00:00 2001 From: Martin Holst Swende Date: Fri, 15 Dec 2017 09:16:03 +0100 Subject: [PATCH] cmd/signer: minor change in json format --- cmd/signer/README.md | 6 +++++- cmd/signer/api.go | 4 ++-- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/cmd/signer/README.md b/cmd/signer/README.md index 43964c730a..f4ec1fc249 100644 --- a/cmd/signer/README.md +++ b/cmd/signer/README.md @@ -434,4 +434,8 @@ invoking methods with the following info: * Address of API (http/ipc) * This makes it posible for the UI to use the api for creating transactions * List of known accounts - \ No newline at end of file + +* The signer should pass the `Origin` header as call-info to the UI. As of right now, the way that info about the request is +put together is a bit of a hack into the http server. This could probably be greatly improved + +* The signer \ No newline at end of file diff --git a/cmd/signer/api.go b/cmd/signer/api.go index f2d5c287f9..9882032889 100644 --- a/cmd/signer/api.go +++ b/cmd/signer/api.go @@ -62,8 +62,8 @@ type ( // SignTxResponse result from SignTxRequest SignTxResponse struct { //The UI may make changes to the TX - Transaction TransactionArg - From common.Address + Transaction TransactionArg `json:"transaction"` + From common.Address `json:"fromaccount"` Approved bool `json:"approved"` Password string `json:"password"` }