peterthomassen / draft-ietf-dnsop-generalized-notify

Other
0 stars 1 forks source link

various updates #15

Closed peterthomassen closed 1 year ago

peterthomassen commented 1 year ago

I agree with almost everything. I disagree with this one:

  • [ ] "types of Notifies and how to interpret them on...": "interpret them in" was better

I don't feel strongly; if you prefer we can change it back. (It seems weird to me, but it may be perfectly fine and just my non-native English telling me that.)

Then there's a couple of more nits, like our inconsistent use of "cds-notificationS.parent" vs "dnskey-notificatioN.child.parent". I suggest we change all of these to be "{RR}-notify...".

Will you create a PR or shall I?