Document how fields in the test vectors have to be interpreted

[stevenroose]

Some binary values are hexadecimal, this can be spotted rather easily. Some, however, appear to be Base64, looking at the code, but they could just as well be UTF8.

Please document this somewhere.

[stevenroose]

Oh, also, is it base64 or base64url? Apparently the modulus and signature fields of a RSA-SHA-256 fulfillment are not valid base64. I'm out of inspiration for testing other encodings.

[stevenroose]

I think I found out it's base64url.

[justmoon]

Yes, this should be documented.

[stevenroose]

Is there a reason why both hexadecimal and base64url are used? I might be OK with making a PR converting them all to the same format, whichever one you prefer. I think hex makes the most sense, it gives you a clear idea about the size of the data and a bit about the contents.

…

On 11 Apr 2017 7:07 PM, "Stefan Thomas" ***@***.***> wrote: Yes, this should be documented. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#10 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AA0F3Eq85lCGjqcIJ1VMm6OcBpOZZHJhks5ru7M6gaJpZM4M5wiJ> .

[justmoon]

Yeah, there is a reason both are used. IIRC, the JSON format for conditions is defined in the cryptoconditions spec and specifies base64url for efficiency.

Outside of the standardized JSON representation of the cryptoconditions I used hex because it makes comparing expected vs actual a lot simpler.

This is from memory, so apologies if I'm misremembering.