Opened 11 years ago

Closed 11 years ago

Last modified 8 years ago

#162 closed design (fixed)

comment ABNF

Reported by: julian.reschke@… Owned by: julian.reschke@…
Priority: normal Milestone: 07
Component: p1-messaging Severity: Active WG Document
Keywords: Cc:

Description

The ABNF in ​http://greenbytes.de/tech/webdav/draft-ietf-httpbis-p1-messaging-06.html#rfc.section.4.2.p.7 for "comment" seems to have the same problem as mentioned in Ticket #31 for "quoted-string". I.e. "ctext" includes the "\" character and thus "(\)" is ambiguous, either "\)" is a "quoted-pair" and the comment is still open or it is a "ctext" backslash followed by ")" thus ending the comment.

Proposed solution: exclude "\" from ctext.

Attachments (2)

162.diff​ (8.7 KB) - added by julian.reschke@… 11 years ago.
proposed change for part 1
162.2.diff​ (8.7 KB) - added by julian.reschke@… 11 years ago.
proposed change for part 1

Download all attachments as: .zip

Change History (9)

comment:1 Changed 11 years ago by julian.reschke@…

  • Owner set to julian.reschke@…
  • Status changed from new to assigned

comment:2 Changed 11 years ago by julian.reschke@…

Also, reported by Brian Smith in ​http://www.w3.org/mid/002201c9cac3$0be2dba0$23a892e0$@org:

Nitpick: The Kleene star is unnecessary and inconsistent with obs-text. This is enough:

ctext = OWS / %x21-27 / %x2A-5B / %x5D-7E / obs-text

Changed 11 years ago by julian.reschke@…

proposed change for part 1

Changed 11 years ago by julian.reschke@…

proposed change for part 1

comment:3 Changed 11 years ago by julian.reschke@…

From [574]:

Disallow \ in ctext, add prose explanation of ABNF, remove unneeded "*" from ctext and qdtext (related to #162)

comment:4 Changed 11 years ago by julian.reschke@…

Applied in [574].

comment:5 Changed 11 years ago by julian.reschke@…

  • Milestone changed from unassigned to 07
  • Priority set to normal

This ticket should be closed after review on the mailing list; sent reminder on 2009-07-27.

comment:6 Changed 11 years ago by julian.reschke@…

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

comment:7 Changed 8 years ago by mnot@…

  • Severity changed from Candidate WG Document to Active WG Document
Note: See TracTickets for help on using tickets.