Seulgi Kim

Do not expose createdAt and updatedAt

2

For instance, The result of `https://corgi.codechain.io/explorer/api/account/wccqxv73gy53dexhzh6qa4v768tw9m7sn7kxvlw6r0a` is ``` {"address":"wccqxv73gy53dexhzh6qa4v768tw9m7sn7kxvlw6r0a","balance":"0","seq":0,"createdAt":"2019-04-10T14:26:20.730Z","updatedAt":"2019-04-15T03:58:35.931Z"} ``` `createdAt` and `updatedAt` are meaningless. It's just an internal time that the data are indexed. Please do not expose them.

I want to find users who have used a specific list of assets. To do this, I need an API returning transfer asset transactions of which the input is in...

Make RlpStream::append() return an error when more than reserved number of items are appended

2

https://github.com/CodeChain-io/codechain/blob/d91b73a0fe81a8a1431e704d9ef73276ef43ef59/util/rlp/src/stream.rs#L397-L409 this tests the behavior. Make the above test pass.

Currently, stake module implements the encoding/decoding for the map. https://github.com/CodeChain-io/codechain/blob/5baa5768d381589ddfc450d774900dcf48cf2bb8/core/src/consensus/stake/action_data.rs#L227 IMO, it's better to move it to the rlp module.

Signature aggregation for block seal

10

Currently, a header has 30 signatures. It's almost 2KB. If we use signature aggregation for seal field, we can reduce the header size sharply.

Currently, all blocks are always verified before importing. However, there is no need to verify the seal when all the peers are trusted nodes. In this case, skipping verification of...

Replace util/io with tokio

1

Currently, the network uses util/io. It returns a token on an event, instead of the related stream. So the network module should manage the relationship between the token and the...

https://github.com/CodeChain-io/codechain/blob/18e1e80af4503cfd4d06e9514e9f88370995abda/sync/src/snapshot/service.rs#L72 Currently, the snapshot service spawns a thread to use the db but doesn't wait for the end of the child thread when it exit. It can corrupt the db.