-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Payload in voucher_status example in A.2 to be updated #50
Comments
See also #47 for more info |
See also anima-wg/anima-bootstrap#144 for related issues in BRSKI. We should capture the outcome of this discussion in the constrained-voucher draft. |
@petervanderstok , how did you create this example? |
By running my implementations , pledge, registrar, masa with the
certficates that are shown.
When changing the certificates, examples text needs to be modified by
hand due to 72 byte line limit.
Peter
Michael Richardson schreef op 2020-11-26 11:56:
@petervanderstok [1] , how did you create this example?
--
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub [2], or unsubscribe [3].
|
mcr
added a commit
that referenced
this issue
Nov 26, 2020
updated voucher_status example, close #50
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In -07, payload decodes to:
{"version": "1", "Status": 1, "Reason": "Informative human readable message", "reason-context": "Additional information"}
which should be (per BRSKI-41):
{"version": "1", "status": false, "reason": "Informative human readable message", "reason-context": { "additional" : "JSON" } }
(status is changed to 'false' here, to encode an error status. If no error, there should be no 'reason' field as well. Also the additional-information is updated with a more clear example, but this is not super important to have. But it's more in line with BRSKI.
Also a success example could be added if needed:
{"version": "1", "status": true}
The text was updated successfully, but these errors were encountered: