ethereumjs/keythereum

package validator

fanatid opened this issue · 1 comments

isHexadecimal looks not good for us validatorjs/validator.js#648
Is keys by geth can be encoded with base64?

should we implement own validators and remove this package?

Good find. It seems simplest to follow the recommendation to use str.length % 2 == 0 && validator.isHexadecimal(str). Or, are there other issues with this library?

Re: base64, no it isn't normally used in Ethereum. I've considered removing base64 support altogether but it's been around a while and someone may be using it, so I've kept it in for now...