Hi Peter,

On 3/10/25 04:46, Peter van Dijk wrote:
If a positive DSYNC answer results,

This appears to be legit English, but it is phrased uncommonly, causing
spurious reboots in human readers :-)

I'm not sure what causes the mental reboot :) Do you have a suggestion for 
improvement?

Perhaps "If the result is a positive DSYNC answer," or closer to your
version "If this results in a positive DSYNC answer,".

Done!

(And perhaps, 4.3 step 1 should be doing more than
flipping the QR bit - it should also remove the report-channel option. This
seems obvious so I wonder if we need to be explicit here.)

NEW
     1.  Acknowledge receipt by sending a NOTIFY response as described in
         [RFC1996] Section 4.7 (identical to NOTIFY query, but with QR bit
         set and any EDNS0 Report-Channel options removed) [...]

Most EDNS options do not want to be mirrored in responses. I think -less-
explicit language is better here. Perhaps just remove the entire bit in
()

Done!

Changes are here: 
https://github.com/peterthomassen/draft-ietf-dnsop-generalized-notify/commit/ffe8308c648ec5ed75d5efed09de1beb9ecd2d64

Cheers,
Peter + co-authors

_______________________________________________
DNSOP mailing list -- dnsop@ietf.org
To unsubscribe send an email to dnsop-le...@ietf.org

Reply via email to