Opened 3 years ago

Closed 21 months ago

#26 closed defect (fixed)

ABNF for pct allows "999"

Reported by: tim@… Owned by: todd.herr@…
Priority: minor Milestone: Deliverable #3 (changes to DMARC base spec + DMARC Usage Guide
Component: dmarc-bis Version:
Severity: - Keywords:
Cc:

Description

The ABBF for the pct tag allows values like "999". The description of the tag mentions "plain-text integer between 0 and 100, inclusive;", but the ABNF allows quite a bit more:

dmarc-percent = "pct" *WSP "=" *WSP

1*3DIGIT

Change History (8)

comment:1 Changed 3 years ago by tim@…

  • Component changed from arc-multi to dmarc-future-notes

comment:2 Changed 3 years ago by freddie@…

Suggestion:

     dmarc-percent   = "pct" *WSP "=" *WSP
                       ( DIGIT / %x31-39 DIGIT / "100" ) 
                       ; 0-100
Last edited 3 years ago by freddie@… (previous) (diff)

comment:3 Changed 21 months ago by todd.herr@…

  • Component changed from dmarc-future-notes to dmarc-bis

comment:4 Changed 21 months ago by todd.herr@…

  • Owner changed from draft-ietf-dmarc-arc-multi@… to todd.herr@…
  • Status changed from new to accepted

In keeping with ticket #47, recommend removing the pct tag entirely, which would include removing the ABNF specification for it.

comment:5 Changed 21 months ago by todd.herr@…

  • Status changed from accepted to started

comment:6 Changed 21 months ago by todd.herr@…

Until Issue #47 is adjudicated, changing ABNF as per suggestion:

  dmarc-percent   = "pct" *WSP "=" *WSP
                    ( DIGIT / %x31-39 DIGIT / "100")
                    ; 0-100
Last edited 21 months ago by todd.herr@… (previous) (diff)

comment:7 Changed 21 months ago by todd.herr@…

Change pushed and merged into main branch

comment:8 Changed 21 months ago by todd.herr@…

  • Resolution set to fixed
  • Status changed from started to closed
Note: See TracTickets for help on using tickets.