Opened 2 years ago
Closed 14 months ago
#47 closed defect (fixed)
Remove pct= tag
Reported by: | seth@… | Owned by: | todd.herr@… |
---|---|---|---|
Priority: | major | Milestone: | Deliverable #3 (changes to DMARC base spec + DMARC Usage Guide |
Component: | dmarc-bis | Version: | |
Severity: | - | Keywords: | nit tag-update |
Cc: |
Description
How pct= is written, how it is implemented, how it is understood, and how it actually works are all different. It's also not statistically sound, and has additional poorly understood real-world behavior when used with p=reject.
Change History (11)
comment:1 Changed 2 years ago by seth@…
- Component changed from rfc7601bis to dmarc-bis
- Owner draft-ietf-dmarc-rfc7601bis@… deleted
- Status changed from new to assigned
comment:2 Changed 18 months ago by todd.herr@…
- Owner set to todd.herr@…
- Status changed from assigned to accepted
comment:3 Changed 18 months ago by todd.herr@…
comment:4 Changed 17 months ago by todd.herr@…
As reported in ticket #5:
Valimail data on 22 March 2021:
74790 DMARC records examined 5548 have pct= tag 5066 of those with pct= defined have pct=100 482 of those with pct= defined have pct= other than 100 53 of those with pct= defined have pct=0
comment:5 Changed 16 months ago by todd.herr@…
- Status changed from accepted to infoneeded
Proposed text pushed to github and merged with main branch.
comment:6 Changed 16 months ago by alex_brotman@…
- Status changed from infoneeded to assigned
I'm not sure I agree with removing pct completely, but seems like 0 and 100 should be ignored? The below is 1 day of data where a pct attribute was present. I removed the long tail of all sorts of broken records.
pct p percent
100 reject 51.194723
100 none 31.497058
100 quarantine 11.766201
10 none 1.680448
50 quarantine 0.750789
5 quarantine 0.373162
10 quarantine 0.343339
10 reject 0.334595
25 quarantine 0.220813
1 quarantine 0.202922
50 reject 0.201107
0 quarantine 0.181093
0 none 0.165156
20 quarantine 0.153728
1 none 0.140622
20 none 0.085060
70 quarantine 0.084914
75 quarantine 0.077796
90 quarantine 0.047547
30 reject 0.047254
75 reject 0.043798
90 reject 0.039404
30 quarantine 0.035075
90 none 0.031865
5 none 0.029756
25 none 0.029535
50 none 0.027966
40 quarantine 0.023774
comment:7 Changed 15 months ago by todd.herr@…
- Status changed from assigned to infoneeded
comment:8 Changed 15 months ago by vesely@…
- Status changed from infoneeded to assigned
Setting p=quarantine; pct=0 seems to be a useful idiom to control From: munging, albeit not so much used.
comment:9 Changed 15 months ago by todd.herr@…
- Status changed from assigned to infoneeded
comment:10 Changed 14 months ago by todd.herr@…
- Status changed from infoneeded to assigned
Consensus from 27 May 2021 interim (https://datatracker.ietf.org/doc/minutes-interim-2021-dmarc-01-202105270900/) was to leave tag in specification but to rewrite its definition to make its implementation and shortcomings more clear.
Rev -02 of dmarcbis does this.
comment:11 Changed 14 months ago by todd.herr@…
- Resolution set to fixed
- Status changed from assigned to closed
See also ticket #5
Elimination of the pct tag will result in removal of the following sections of text:
From "DMARC URIs":
From "Formal Definition":
and
The entire "Message Sampling" section:
From "DMARC Tag Registry":
In "Entire Domain, Monitoring Only", change the following:
to
or perhaps remove the bullet entirely since it adds no value without the mention of the pct tag
Last, rewrite the "Subdomain, Sampling, and Multiple Aggregate Reports URIs" section to remove mention of sampling and the use of the pct tag.