Opened 9 years ago

Closed 9 years ago

#14 closed enhancement (fixed)

Itsuma Tanaka Review of draft-ietf-dime-extended-naptr-02

Reported by: tanakai@… Owned by: tanakai@…
Priority: minor Milestone: milestone1
Component: extended-naptr Version: 1.0
Severity: In WG Last Call Keywords:


I have reviewed draft-ietf-dime-extended-naptr-02. Let me provide some (mainly editorial) comments.

[Comment 1] Introduction section should have a brief problem statement (i.e. what the problem this is trying to solve) so that the readability is improved.

[Comment 2] Editorial in Section 3, "The DNS administrator of some domain SHOULD also provision base RFC 3588 style..."

shouldn't this be

"The DNS administrator of some domain SHOULD also provision *legacy* RFC 3588 style...".. ?

My proposal is to replace 'base' with 'legacy', so that the consistent expression is used throughout the specification.

[Comment 3] Editorial in Section 4, the first sentence. "The basic Diameter Peer Discover principles" should be "The basic Diameter Peer Discovery principles"

Change History (1)

comment:1 Changed 9 years ago by jouni.nospam@…

  • Resolution set to fixed
  • Status changed from new to closed

Fixed in -05.

Note: See TracTickets for help on using tickets.