mlsecproject / combine

Tool to gather Threat Intelligence indicators from publicly available sources
https://www.mlsecproject.org/
GNU General Public License v3.0
655 stars 171 forks source link

Move the combine.cfg file to a combine-default.cfg #68

Closed alexcpsec closed 10 years ago

alexcpsec commented 10 years ago

So we can have a standard .cfg file and people (including myself) can edit their config files with their information without fear of commiting their API key (like I almost did right now).

alexcpsec commented 10 years ago

@technoskald do you have any input on this? Is this really the best way to get this done? Is this a recognizable pattern in tools like this?

krmaxwell commented 10 years ago

So the first and second points are common (and trivial). But I would do it as:

alexcpsec commented 10 years ago

Agree. Want me to do this, or can you take care of it?

krmaxwell commented 10 years ago

I will work on it.