-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add constant padding to random hexes, hashes and addresses #494
Add constant padding to random hexes, hashes and addresses #494
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
…ve-a-common-prefix
📦 Size Report
|
@@ -19,10 +19,11 @@ export function CheckedAddress(value: string): CheckedAddress { | |||
* Generates a random address. Tries to represent desired ascii prefix as hex value. Helps to identify transactions in the logs. | |||
*/ | |||
CheckedAddress.random = (asciiPrefix = ''): CheckedAddress => { | |||
const constantAddressPrefix = '0000' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i thought about this more: i think a good prefix is deadbeef as it's obviously not random. Or simply make the prefix longer - for example: 8 chars instead of 4.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changed to deadbeef
@@ -19,10 +19,11 @@ export function CheckedAddress(value: string): CheckedAddress { | |||
* Generates a random address. Tries to represent desired ascii prefix as hex value. Helps to identify transactions in the logs. | |||
*/ | |||
CheckedAddress.random = (asciiPrefix = ''): CheckedAddress => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess it will be infix
now, since it's not first part of the address anymore.
I'm curious, why this change was needed in the first place?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know if we need to be that precise to rename that to infix
- both of these parts are still pre
randomPart 😛
…ve-a-common-prefix
…ve-a-common-prefix
No description provided.