-
Notifications
You must be signed in to change notification settings - Fork 51
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
Multiple syntax issues in config file #12
Comments
I haven't had to use this tool since I wrote it, but I'll try my best. What platform are you using? What version of Python? |
Yeah I just noticed when the last commit was. I probably wouldn't have added this issue if I noticed that beforehand tbh :) |
It's no problem. It's probably something simple. I wonder if it has something to do with the underscore in the value name? https://github.com/keithjjones/hostintel/blob/master/sampleconfig.conf#L2 That's the only one that has one in that config file. You could test it by changing the code and the config file. |
This is where it is used... |
Cool, thanks for the info. I'll do some debugging and get back to you - not super urgent on my end so no worries. If I figure out a solution I'll just post it here and close out the ticket |
Putting certain config values in quotes fixed in the syntax error problem, now I'm hitting this:
It very much does exist though. This is turning into a rabbit hole :p |
A recent issue was posted here with configparser. I wonder if it's related. |
Any developments with this one? |
I actually haven't looked at this recently since my last comment much, I'll check it out again this weekend and send an update. I'll likely re-clone the repo and start from scratch in a venv just to make sure nothing external interferes |
It seems there's multiple issues with either the config file or its use with ConfigParser (or the ConfigParser version?)
Using the default config and editing it to add my keys throws this exception:
I've tried this using just
data/GeoLite2-City.mmdb
and it has the same problem. Won't run at all due to the vague Exception of "invalid syntax". I think pinning the version of ConfigParser in autorequirements.txt would help since typically ConfigParser takes=
instead of:
, at least from my experience. I changed this myself in the config but hit another exception with my username for PassiveTotalPutting quotes around my email address then yields this exception:
Any help here would be greatly appreciated 👍
The text was updated successfully, but these errors were encountered: