raiden-contracts icon indicating copy to clipboard operation
raiden-contracts copied to clipboard

Raiden Network Smart Contracts

Results 100 raiden-contracts issues
Sort by recently updated
recently updated
newest added

`ChannelOpened` uses the order of the addresses provided by the caller: https://github.com/raiden-network/raiden-contracts/blob/7212a66be1908af8b657c36de5393a04838af2e4/raiden_contracts/data/source/raiden/TokenNetwork.sol#L301-L306 `ChannelSettled` uses the a very weird order. The participant which has the highest transferred amount will be the...

P2
efforts: a few hours hack

Since there are multiple tokens involved, a more specific option name might prevent mistakes.

efforts: low_hanging_fruit
P3

Currently, `BALANCE_PROOF` is used to close a channel, and `BALANCE_PROOF_UPDATE` is used to submit counter balance proofs. I think there are better names. `BALANCE_PROOF_CLOSE` and `BALANCE_PROOF_UPDATE` probably.

P3

Currently, the contract ABI and the contract address are provided in two different function calls. The ABI comes from `contracts_precompiled_path()` and the deployment comes from `get_contracts_deployment_info()`. When the user gives...

needs discussion
P2
needs discussion - Raiden Client
needs discussion - MS/PFS
efforts: a few days hack

A CLI tool that can call `ServiceRegistry.change_parameters()` might be useful.

P3
efforts: a few hours hack

`balance_hash`'s are meant to hide concrete transferred amounts from MonitoringServices, but in the current form, they have not enough entropy so that an observer can easily recover the contents of...

P2
efforts: a few hours hack

The offchain withdraw proofs can be contained in a withdraw request message or in a withdraw message. In order to distinguish these two kinds in the signature checking, the signature...

efforts: low_hanging_fruit
P2

@jomuel noticed ``` three different signatures in use: mint(address to, uint256 value) returns (bool) (tokenmint.io) increaseSupply(uint256 value, address to) (EIP 621) mintFor(uint256 amount, address target) (Raiden testnet token) The endpoint...

P4: good_to_have
efforts: a few hours hack

Currently `ContractVerifier` exposes interfaces for 1) checking all Raiden contract deployments and 2) checking all service contract deployments. However, in `raiden` client code, each proxy only needs to check one...

P3
efforts: a few days hack

Since #1023 was merged, it has become harder to recognize the version from a Solidity source. Perhaps each file can contain an ever-increasing integer. My biggest wonder is how not...

P2