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

Different category in typeDefinition / defaultCategory #3119

Merged
merged 1 commit into from
Mar 29, 2018

Conversation

3c7
Copy link
Contributor

@3c7 3c7 commented Mar 28, 2018

Assigned "Network activity" as default category for url in $typeDefinitions as defined in $defaultCategories. There were two different "default categories" used. "External analysis" is skipped by external tools because of normally referencing to reports etc. by external parties.

  • Used on testing env.
  • Patch

@3c7
Copy link
Contributor Author

3c7 commented Mar 28, 2018

Ouh shit. Breaking tests. So External analysis is expected to be the default value for category in the tests. That results in Feeds using External analysis -> urls in e.g. URLHaus feed not properly blocked by external tools. Lots of OSINT events also use External analysis->url for referring to reports instead of using the link datatype and because of that all External analysis attributes are getting skipped in order not to block websites of analysts and so on.

Views & opinions on this?

@iglocska
Copy link
Member

Ah ok, completely misunderstood this. Basically $typeDefinitions / $defaultCategories isn't always used when it comes to setting defaults. What you're after is the freetext parser's algorithm - just search for complextypetool (it's in /var/www/MISP/app/Lib/Tools/ComplexTypeTool.php IIRC)

@3c7
Copy link
Contributor Author

3c7 commented Mar 29, 2018

I'm using this in production right now to be able to automatially use some feeds for enhancing some internal blocklists. Changing $typeDefinitions['url']['default_category'] fixed the problem for us. Basically I just wanted to know, why the default for url is External Analysis in MISP and maybe start a discussion to change that as this makes not too much sense to me. But maybe I'm not getting the idea behind that. ;)

@iglocska
Copy link
Member

indeed, the default for url should be Network activity. External analysis is just plain wrong - it should be that for "link", but definitely not url.

@3c7
Copy link
Contributor Author

3c7 commented Mar 29, 2018

Okay, so I create a PR for PyMISP to change it there as well?

3c7 added a commit to 3c7/PyMISP that referenced this pull request Mar 29, 2018
@Rafiot Rafiot merged commit 628d46d into MISP:2.4 Mar 29, 2018
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