Skip to content

small CRITs update and maxmind#94

Merged
krmaxwell merged 5 commits intomlsecproject:masterfrom
paulpc:master
Dec 17, 2014
Merged

small CRITs update and maxmind#94
krmaxwell merged 5 commits intomlsecproject:masterfrom
paulpc:master

Conversation

@paulpc
Copy link
Contributor

@paulpc paulpc commented Dec 17, 2014

Small updates to the list of error codes to allow for older CRITs versions (include accepting 400 status codes).
Furthermore, added a script to download the maxmind databases (should take care of #24)?

@alexcpsec
Copy link
Member

Hey, Paul! Thanks for the script and fix!

Do you mind separating out the globe visualizer part on a different PR? If was to add maps support to combine, I would be partial to https://github.com/hrbrmstr/pewpew ;)

@paulpc
Copy link
Contributor Author

paulpc commented Dec 17, 2014

my bad - i should have done the globe on another branch

@paulpc
Copy link
Contributor Author

paulpc commented Dec 17, 2014

there you go! Sorry about that - i thought git would have taken a snapshot of the code when i issued the PR - now i know to use branches

@alexcpsec
Copy link
Member

Heh, thanks! When you have a PR for the globe thing we can discuss that further.

@krmaxwell krmaxwell merged commit c0c50c3 into mlsecproject:master Dec 17, 2014
@alexcpsec alexcpsec removed the unknown label Dec 17, 2014
@alexcpsec
Copy link
Member

Thanks for the assist @technoskald !

@krmaxwell
Copy link
Member

💪

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

Successfully merging this pull request may close these issues.

3 participants