mattrglobal / jwm

JSON Web Message (JWM) RFC
16 stars 6 forks source link

reply_to property value definition seems wrong #54

Closed kdenhartog closed 4 years ago

kdenhartog commented 4 years ago

... is a case-sensitive string containing an array of case-sensitive strings ...

This seems incorrect. It would lead me to believe this is what the spec is after

{
    "reply_to": "["didUrl1", "didUrl2"]"
}

Rather I think it intends for the json to look like this:

{
    "reply_to": ["didUrl1", "didUrl2"]
}

Which means the text should be updated to say this:

... is an array of case-sensitive strings ...

tplooker commented 4 years ago

This issued was fixed in #55