Re: [saag] AD Sponsorship of draft-housley-ers-asn1-modules

[Sean Turner <[e-mail]>]

> On Jun 10, 2021, at 13:31, Carl Wallace <[e-mail]> wrote:
> 
> Inline...
> 
> On 6/9/21, 11:03 AM, "saag on behalf of Sean Turner" <[e-mail] on behalf of [e-mail]> wrote:
> 
>    Roman,
> 
>    I have but one point to raise and then some cosmetic nits (cosmetic because compilers ignore whitespace).
> 
>    0) Point to Raise:
> 
>    re: AllWantBacks. I am not entirely sure whether what is there for swb-ers-all WANT-BACK merely defines the new value or whether it also adds it to the list of available AllWantBacks.  AllWantBacks is imported from RFC 5912:
> 
>    AllWantBacks WANT-BACK ::= {
>         WantBackSet | ACertWantBackSet | AnyWantBackSet, ...
>     }
> 
>    To add swb-ers-all to the list, I wonder whether merely defining it is enough. Is there something more that needs to be done to get it into the list as the fourth option?
> 
> [CW] The swb-ers-all does not impact AllWantBacks. It was simply a shorthand means of saying "I want an EvidenceRecord for everything in replyWantBacks". See the last paragraph of section 3 in RFC5276. The ExpandedWantBacks in this draft is a superset of AllWantBacks. Unless I am missing something, I do not see a concern. 
> 
> <snip>

Ah! I see it now - you’re not expanding AllWantBacks. Not sure why I got it in my head that you were trying to expand AllWantsBacks. lgtm now.

spt