msimerson / mail-dmarc

Mail::DMARC, a complete DMARC implementation in Perl
Other
33 stars 23 forks source link

Use of uninitialized value in dmarc_send_reports and Aggregate.pm #156

Closed freddieleeman closed 3 years ago

freddieleeman commented 4 years ago

Have seen a couple of these while logging version v1.20191004:

Domain: <hidden>
Use of uninitialized value in concatenation (.) or string at /usr/local/bin/dmarc_send_reports line 105.
rua:
Use of uninitialized value $reco in concatenation (.) or string at /usr/local/share/perl/5.28.1/Mail/DMARC/Report/Aggregate.pm line 65.
No valid ruas found, deleting report

Domain: <hidden>
rua:
No valid ruas found, deleting report
sleeping 5.Use of uninitialized value in concatenation (.) or string at /usr/local/bin/dmarc_send_reports line 105.
Use of uninitialized value $reco in concatenation (.) or string at /usr/local/share/perl/5.28.1/Mail/DMARC/Report/Aggregate.pm line 65.

The rua: value in the verbose output appears to be empty but the domains do have a rua tag specified.

msimerson commented 4 years ago

The rua: value in the verbose output appears to be empty but the domains do have a rua tag specified.

The RUA may be specified, but is it syntactically correct? If it fails to parse, that would result in the error you're seeing.

freddieleeman commented 4 years ago

I seem to have misplaced the original log and am unable to validate the original rua value. I will however keep an eye out if I ever notice the same error again and will then validate the rua value. We notice a lot of people publish a DMARC record without the 'mailto:' prefix in the rua value. Maybe that was the case in this situation as well.

freddieleeman commented 3 years ago

Haven't seen this issue in a long time. Closing