Skip to content
This repository has been archived by the owner on Jun 6, 2019. It is now read-only.

add sanity check on the returned unsigned transaction #22

Open
evq opened this issue Nov 9, 2017 · 0 comments
Open

add sanity check on the returned unsigned transaction #22

evq opened this issue Nov 9, 2017 · 0 comments

Comments

@evq
Copy link
Contributor

evq commented Nov 9, 2017

currently the bat-client blindly signs the transaction that is returned from the ledger server. better to be defensive and check that the amount matches expectations (<= self.state.properties.fee.amount)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant