authindicators / rfc-brand-indicators-for-message-identification

Other
20 stars 6 forks source link

ABNF Upates #23

Closed toddherr closed 2 years ago

toddherr commented 2 years ago

To correct some errors and to make the ABNF generally more readable, in keeping with similar formatting in DMARCbis, the following ABNF updates are proposed for section 4.2, Assertion Record Definition:

Change bimi-version = %x76 *WSP "=" *WSP %x42.49.4d.49 1DIGIT to bimi-version = "v" *WSP "=" *WSP "BIMI1"

; The purpose of this one is to make it clear that bimi-uri is optional with the 'a' tag; that is, "a=" by itself is a valid declaration Change bimi-evidence-location = %x61 *WSP "=" bimi-uri to bimi-evidence-location = "a" *WSP "=" *WSP [bimi-uri]

; The purpose of this one is to make it clear that bimi-uri is optional with the 'l' tag; that is, "l=" by itself is a valid declaration, especially as shown in Section 4.2.1, Declination to Publish Change bimi-location = %x6c *WSP "=" bimi-uri to bimi-location = "l" *WSP "=" *WSP [bimi-uri]

Change bimi-sep = *WSP %x3b *WSP to bimi-sep = *WSP ";" *WSP

; The purpose of this one is to make it clear that the entire 'a' tag is optional. Change bimi-record = bimi-version (bimi-sep bimi-location) (bimi-sep bimi-evidence-location) \[bimi-sep\] to bimi-record = bimi-version (bimi-sep bimi-location) [(bimi-sep bimi-evidence-location)] [bimi-sep]

toddherr commented 2 years ago

I think this is the complete list of valid BIMI records:

v=BIMI1; l= v=BIMI1; l=; v=BIMI1; l=http://full/path/to/svg/file v=BIMI1; l=http://full/path/to/svg/file; v=BIMI1; l=; a= v=BIMI1; l=; a=; v=BIMI1; l=http://full/path/to/svg/file; a= v=BIMI1; l=http://full/path/to/svg/file; a=; v=BIMI1; l=http://full/path/to/svg/file; a=http://full/path/to/evidence/document.pem v=BIMI1; l=http://full/path/to/svg/file; a=http://full/path/to/evidence/document.pem;