Jude Nelson comments

Results 324 comments of


Jude Nelson

Improved Atlas shutdown and crash tolerance (#3082)

This needs to be looked at.

[Clarity] Compress contracts before storing them and decompress on load, and bill the user only for the number of bytes in the compressed representation

This is something we'd like to do in the near future. @cylewitruk has graciously taken on the implementation effort.

[Clarity] Compress contracts before storing them and decompress on load, and bill the user only for the number of bytes in the compressed representation

Assigning to @obycode for now. Please feel free to re-assign.

Improve/fix `tip` parameter

Closing due to inactivity.

VM: analysis pass should consider argument count when deploying a contract

Transactions with the wrong number of arguments will be mined in 2.1 because we convert all analysis errors to runtime errors.

VM: analysis pass should consider argument count when deploying a contract

Assigning to @obycode so this issue has an owner. Please feel free to re-assign.

Contract name length for Clarity 2

> The parser can't do anything to prevent a contract from being published with a long name, or passing a contract principal with a long name as a contract call...

Contract name length for Clarity 2

@zone117x I wish I saw that message earlier and comprehended its significance. I just noticed that my 2.1 node does not boot. In fact, it gets stuck when trying to...

Contract name length for Clarity 2

The bug isn't in the parser. It's in 2da31c5c99, where you changed `CONTRACT_NAME_REGEX_STRING` in `clarity/src/vm/representations.rs` to only accept strings of length at most `CONTRACT_MAX_NAME_LENGTH`.

Contract name length for Clarity 2

No worries! This is why we're doing the genesis test before -rc1 -- to shake out any inconsistencies like this :) I'll keep my eye on the node and see...