Limits name length of claimtrie transactions to 255 bytes #23

Merged
kaykurokawa merged 1 commit from max_name_length_fix into master 2016-07-21 21:25:40 +02:00
kaykurokawa commented 2016-07-21 20:16:44 +02:00 (Migrated from github.com)

This patch limits name length of claimtrie transactions to 255 bytes at most.
This is a soft fork, and nodes should upgrade immediately.

This patch limits name length of claimtrie transactions to 255 bytes at most. This is a soft fork, and nodes should upgrade immediately.
jobevers commented 2016-07-21 20:48:58 +02:00 (Migrated from github.com)

I don't know c++ or the lbrycrd code well enough to give a thumbs up so will defer that to somebody else.

I don't know c++ or the lbrycrd code well enough to give a thumbs up so will defer that to somebody else.
kaykurokawa commented 2016-07-21 21:02:52 +02:00 (Migrated from github.com)

fixed comments, and squashed. @jimmykiselak gave a review and Ok on slack.

fixed comments, and squashed. @jimmykiselak gave a review and Ok on slack.
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/lbrycrd#23
No description provided.