search

mlsecproject / combine

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

Logging + Updated grequests + exception handling (closes #78, closes #32, closes #34) #83

Closed alexcpsec closed 9 years ago

alexcpsec commented 9 years ago

This is is the final PR merge candidate to me, replacing #81 .

Waiting any comments or :+1: before I merge

alexcpsec commented 9 years ago

@technoskald

Please review https://github.com/mlsecproject/combine/issues/78 and https://github.com/kennethreitz/grequests/pull/58 for more details on this

krmaxwell commented 9 years ago

@alexcpsec Thanks, looking at them now. I'm hoping we can find a more elegant stopgap solution.

krmaxwell commented 9 years ago

I guess this is good enough for now. Wonder if we can pressure the grequests folks to merge that fix soon (I see @gbrindisi is already trying).