Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Nameservers parameter typo in config file causes "Not a valid report" error on processing reports #543

Closed
jovimon opened this issue Aug 20, 2024 · 1 comment

Comments

@jovimon
Copy link

jovimon commented Aug 20, 2024

Hello,
I've upgraded to 8.12.0 and had a typo on the nameserver parameter in config file, which looked like this (notice the IP address ending in 333):

nameserver = 10.10.10.1,10.10.10.2,10.10.10.333,10.10.10.4

This typo caused parsedmarc to not recognize any of the reports given to it.

INFO:cli.py:1023:Starting parsedmarc
DEBUG:__init__.py:1244:Parsing whatever.xml.gz
ERROR:cli.py:1200:Failed to parse whatever.xml.gz - Not a valid report

If I correct the typo, without any other change to the config file or the reports, and they get processed without any problem:

INFO:cli.py:1023:Starting parsedmarc
DEBUG:__init__.py:1244:Parsing whatever.xml.gz
WARNING:utils.py:165:get_reverse_dns(AAA.BBB.CCC.DDD) exception: The DNS query name does not exist: DDD.CCC.BBB.AAA.in-addr.arpa.
DEBUG:utils.py:422:IP address AAA.BBB.CCC.DDD reverse_dns not found
INFO:elastic.py:369:Saving aggregate report to Elasticsearch
WARNING:cli.py:100:An aggregate report ID 282692$6727846=00b96be3def2a634 from example.com about mydomain.org with a date range of 2023-11-16 23:00:03Z UTC to 2023-11-17 23:00:02Z UTC already exists in Elasticsearch

Thank you very much.

spaolo added a commit to spaolo/parsedmarc that referenced this issue Sep 18, 2024
spaolo added a commit to spaolo/parsedmarc that referenced this issue Sep 18, 2024
@jovimon
Copy link
Author

jovimon commented Sep 24, 2024

Thank you very much for your help @spaolo . Hope it gets merged soon.

seanthegeek added a commit that referenced this issue Oct 3, 2024
- Proper IMAP namespace fix (Closes issue #557 and issue #563)
  - Require `mailsuite>=1.9.17`
  - Revert PR #552
- Add pre-flight check for nameservers (PR #562 closes issue #543)
- Reformat code with `ruff`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants