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

ASN Lookup expert: Missing sane default value for database path #2566

Open
sebix opened this issue Feb 14, 2025 · 0 comments
Open

ASN Lookup expert: Missing sane default value for database path #2566

sebix opened this issue Feb 14, 2025 · 0 comments

Comments

@sebix
Copy link
Member

sebix commented Feb 14, 2025

database = None # TODO: should be pathlib.Path

I suggest that the default path is /var/lib/intelmq/bots/asn_lookup/ipasn.dat as was the case some time ago.

Also, the update script could check first if the path exists (and isn't a None or empty string) and even if it is writeable before downloading the data.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant