On Wed, 23 May 2018, RW wrote:

On Tue, 22 May 2018 10:53:40 -0700 (PDT)
John Hardin wrote:

On Tue, 22 May 2018, Benny Pedersen wrote:

header     __REPTO_MULTI_ADDR         Reply-To:address =~ /,/

imho its

header     __REPTO_MULTI_ADDR         Reply-To:addr =~ /,/

I have a fairly thorough lint check that disables various plugins
that I run before committing changes. It passes lint here...

That said, fixing for consistency.

It's more than cosmetic.  From a quick look at the code I think that
even though it passes lint, :address would just be ignored.

The rule would still work most of the time, but would FP occasionally,
e.g. on display names like "Smith, John".

Yeah, after some testing I found that you can't rely on lint to catch that. It happily accepts anything (like ":fnord"). :(

Opening a boog.


--
 John Hardin KA7OHZ                    http://www.impsec.org/~jhardin/
 [email protected]    FALaholic #11174     pgpk -a [email protected]
 key: 0xB8732E79 -- 2D8C 34F4 6411 F507 136C  AF76 D822 E6E6 B873 2E79
-----------------------------------------------------------------------
  The third basic rule of firearms safety:
  Keep your booger hook off the bang switch!
-----------------------------------------------------------------------
 418 days since the first commercial re-flight of an orbital booster (SpaceX)

Reply via email to