status-network-token icon indicating copy to clipboard operation
status-network-token copied to clipboard

Do another round of auditing.

Open MicahZoltu opened this issue 8 years ago • 1 comments
trafficstars

There have been substantial changes to the contracts since the audits took place. You can see the full diffs here for each audit relative to current master: https://github.com/status-im/status-network-token/compare/2152b17aa2ef584a2aea95533c707a345c6ccf69...master https://github.com/status-im/status-network-token/compare/ef163f1b6fd6fb0630a4b8c78d3b706f3fe1da33...master

Included in these changes are fairly significant changes to the contribution contract and the dynamic ceiling contract. These changes go far beyond minor bug fixes and addressing auditor concerns.

On your blog you have stated:

Independent audits suggest there are no issues that need resolving

Which I believe to be a bit disingenuous due to the volume of changes that have occurred since the audits took place.

MicahZoltu avatar Jun 17 '17 17:06 MicahZoltu

Your point is valid, and the line has been redacted.

Majority of the changes have been to improve code quality and strictness in both code and logic.

The logic that has changed is in DynamicCeiling.sol and we've gone over it many times.

We do have a bug bounty https://blog.status.im/status-network-token-bug-bounty-a66fc2324359

Please help us out :)

0xc1c4da avatar Jun 18 '17 05:06 0xc1c4da