Consensys / Consensys/eth-lightwallet

Support standardized Hex format(s)

Open
#49 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
1.5k
Forks
513
PR merge metrics
No merged PRs in 30d

Description

There's 2 commons hex formats at the moment
- prefixed hex strings `0x123456`
- Buffers `new Buffer('123456', 'hex')` ( ideal memory footprint )

Currently `eth-lightwallet` expects non-prefixed strings.
Maybe wrap all public fn inputs in a format normalizer fn

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the public functions in eth-lightwallet that currently expect non-prefixed hex strings and trace how their inputs are handled. Done means the relevant public inputs accept both prefixed hex strings and hex Buffers without breaking existing non-prefixed string usage; check the existing test suite for coverage.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
cryptography
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.