MissingFieldError("crypto"): should check for "Crypto" too

[etcdevteam]

MissingFieldError("crypto"): should check for "Crypto" too

[r8d8]

@whilei , actually we can't handle this because JSON deserialization is a case sensitive, it counts crypto and Crypto as 2 different fields.

[splix]

@r8d8 but you can convert it to lower case before parsing json

[r8d8]

It will require reqexp, I just can't convert whole file (case in nameand description is up to user )
Ok, I think it wouldn't reduce perfomance too much

[whilei]

Can just check Crypto and crypto instead of only one JSON key? So if crypto doesn't exist check for Crypto or vice versa?

I ask because the only discrepancy I've personally run into is C/crytpo... might not be worth trying to do "global" case-insensitivity?

[r8d8]

yeap, but even not greedy regexp will parse half of json till it reaches crypto
Lets make it and solve any bottleneck later, if they will occur at all.

[splix]

maybe there is a way to use a simple string replacement, w/o regexp? anyway, it wouldn't be a bottleneck, as it used only once during initial import