Skip to content
This repository was archived by the owner on Mar 1, 2019. It is now read-only.

Address Discrimination Not Checked When Sending Transactions #94

Closed
KtorZ opened this issue Nov 23, 2018 · 0 comments
Closed

Address Discrimination Not Checked When Sending Transactions #94

KtorZ opened this issue Nov 23, 2018 · 0 comments
Labels
BUG Something isn't working
Milestone

Comments

@KtorZ
Copy link
Contributor

KtorZ commented Nov 23, 2018

Context

Commit Hash / Release Tag input-output-hk/cardano-ls@release/2.0.0
Operating System (Windows, OSX, Linux)

On release/2.0.0, one can attempt to construct and send transactions to addresses which are of a different network via the wallet-new V1 API. Even though the nodes reject this transaction since it's invalid (as a result of CO-410), an error should be returned by the API in order to prevent users from even attempting this.

Steps to Reproduce

  1. Manually craft a Cardano Address using a wrong protocol magic
  2. Use this address in a transaction

Expected behavior

The wallet backend API should reject this transaction request.

Actual behavior

Transaction is accepted by the wallet, and later appears as failed when rejected by the node.


PR

Number Base
input-output-hk/cardano-sl#3823 develop

Retrospective

This isn't the first discrepancy we notice between nodes and wallet in terms of performed checks (cf the fee calculation round vs ceiling ...) and it's probably worth investigating further to catch potential bugs.

@KtorZ KtorZ added the BUG Something isn't working label Nov 23, 2018
@KtorZ KtorZ added this to the Bugs (Release 2.0.0) milestone Nov 23, 2018
@KtorZ KtorZ closed this as completed Nov 23, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
BUG Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant