Closed b-kamphorst closed 4 months ago
Ok. Will review and likely merge this in, in next day. Thank you
On Tue, Jun 25, 2024, 5:19 AM bart @.***> wrote:
Hi @dhubler https://github.com/dhubler,
While interacting with the FreeCONF server, we noted some issues in the response to /.well-known/host-meta. Specifically, it did not meet the specification of RFC 7033 https://datatracker.ietf.org/doc/html/rfc7033#section-4.4:
- incorrect top-level object xrd
- missing required subject member
- incorrect definition of links (should be an array with members)
- incorrect @ in keys
See also https://www.packetizer.com/json/jrd/ and the example in https://www.rfc-editor.org/rfc/rfc6415#appendix-A.
Please consider this PR :)
You can view, comment on, or merge this pull request online at:
https://github.com/freeconf/restconf/pull/64 Commit Summary
- 623525d https://github.com/freeconf/restconf/pull/64/commits/623525d4184953e865db42cd1b4c7f6fcd9f01c1 fix: correct response to .well-known/host-meta (RFC 7033)
File Changes
(2 files https://github.com/freeconf/restconf/pull/64/files)
- M server.go https://github.com/freeconf/restconf/pull/64/files#diff-366e46a40f6f60b4f7614eb0976bb51820364bf5ca6ccc4787eb49d7bdbef3e6 (3)
- M testdata/gold/well-known https://github.com/freeconf/restconf/pull/64/files#diff-1f3c4b1c98686edaa456f2296486528253c45d113b5f8b6b1efb010381c496ab (2)
Patch Links:
- https://github.com/freeconf/restconf/pull/64.patch
- https://github.com/freeconf/restconf/pull/64.diff
— Reply to this email directly, view it on GitHub https://github.com/freeconf/restconf/pull/64, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAACA7RQ7EBGYJRN7KOLBWTZJEY3JAVCNFSM6AAAAABJ3NP2UWVHI2DSMVQWIX3LMV43ASLTON2WKOZSGM3TEMJVGMZTCNY . You are receiving this because you were mentioned.Message ID: @.***>
Hi @dhubler,
While interacting with the FreeCONF server, we noted some issues in the response to
/.well-known/host-meta
. Specifically, it did not meet the specification of RFC 7033:xrd
subject
memberlinks
(should be an array with members)@
in keysSee also https://www.packetizer.com/json/jrd/ and the example in https://www.rfc-editor.org/rfc/rfc6415#appendix-A.
Please consider this PR :)