search

ietf-rats / ietf-corim-cddl

This repository is abandoned. The adopted I-D can be found at:
https://github.com/ietf-rats-wg/draft-ietf-rats-corim/
2 stars 0 forks source link

align xcorim layout with recent changes to corim #137

Closed thomas-fossati closed 3 years ago

thomas-fossati commented 3 years ago

Fixes #136, Fixes #135

nedmsmith commented 3 years ago

protected-signed-xcorim-header-map = {

Should this be changed to: protected-xcorim-header-map = {

yogeshbdeshpande commented 3 years ago

protected-signed-xcorim-header-map = {

Should this be changed to: protected-xcorim-header-map = {

No, the existing name is correct. This protected section is part of xcorim signature.

thomas-fossati commented 3 years ago

protected-signed-xcorim-header-map = { Should this be changed to: protected-xcorim-header-map = {

No, the existing name is correct. This protected section is part of xcorim signature.

true, though TBH a bit pleonastic...

yogeshbdeshpande commented 3 years ago

pleonastic

No, its not, in fact it is necessary, given the fact that part of the structure elements (fields) are not part of signature and part does have signature over them!

thomas-fossati commented 3 years ago

pleonastic

No, its not, in fact it is necessary, given the fact that part of the structure elements (fields) are not part of signature and part does have signature over them!

"protected" and "signed" are nearly synonyms in this case... anyway, mine was just a fly-by note, I have no stakes in this matter :-)