domainaware / checkdmarc

A parser for SPF and DMARC DNS records
https://domainaware.github.io/checkdmarc
Apache License 2.0
239 stars 75 forks source link

export to csv fails when BIMI flag is included #126

Open EmailKarma opened 5 months ago

EmailKarma commented 5 months ago

I believe part of this is related to the "fields" list in init.py missing the [BIMI values] for the CSV output. I looked but couldn't figure out where the missing values originate from to attempt to make adjustments.

checkdmarc --skip-tls example.com -b default -n 9.9.9.9 -o c:\temp\example.csv

Traceback (most recent call last): File "", line 198, in _run_module_as_main File "", line 88, in _run_code File "C:\Users\redacted\AppData\Local\Programs\Python\Python312\Scripts\checkdmarc.exe__main.py", line 7, in File "C:\Users\redacted\AppData\Local\Programs\Python\Python312\Lib\site-packages\checkdmarc_cli.py", line 122, in _main output_to_file(path, results_to_csv(results)) ^^^^^^^^^^^^^^^^^^^^^^^ File "C:\Users\redacted\AppData\Local\Programs\Python\Python312\Lib\site-packages\checkdmarc__init__.py", line 388, in results_to_csv rows = results_to_csv_rows(results) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "C:\Users\redacted\AppData\Local\Programs\Python\Python312\Lib\site-packages\checkdmarc\init__.py", line 273, in results_to_csv_rows row["bimi_warnings"] = "|".join(_bimi["warnings"])


KeyError: 'warnings'
EmailKarma commented 5 months ago

I realize it would be helpful to mention I'm running v 5.3.1