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

Fortiguard url taxonomy #296

Merged
merged 4 commits into from
Sep 3, 2018
Merged

Fortiguard url taxonomy #296

merged 4 commits into from
Sep 3, 2018

Conversation

srilumpa
Copy link
Contributor

@srilumpa srilumpa commented Jul 5, 2018

This addresses #295 by allowing customization of the malicious and suspicious categories through the Cortex configuration.

Instead of having Malicious Websites hard-coded as the only malicious category and Suspicious Websites hard-coded as the only suspicious category (which, by the way, is not listed anymore as a FortiGuard URL category), the list can be configured in Cortex. This will allow more flexibility to users to define what is malicious and what is suspicious according to their company/internal/personal Security Policy. Every category not listed in those two lists will be considered as safe.

Beware, the comparisons are, for now, case sensitive.

The full list of categories can be found at https://fortiguard.com/webfilter/categories.

@jeromeleonard jeromeleonard added scope:analyzer Issue is analyzer related category:enhancement Issue is related to an existing feature to improve labels Jul 16, 2018
@jeromeleonard jeromeleonard added this to the 1.12.0 milestone Jul 16, 2018
@jeromeleonard jeromeleonard self-requested a review July 27, 2018 06:02
@jeromeleonard jeromeleonard self-assigned this Sep 3, 2018
@jeromeleonard jeromeleonard merged commit bfe9cf0 into TheHive-Project:develop Sep 3, 2018
jeromeleonard added a commit that referenced this pull request Sep 3, 2018
@srilumpa srilumpa deleted the fortiguatd_url_taxonomy branch June 4, 2019 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:enhancement Issue is related to an existing feature to improve scope:analyzer Issue is analyzer related status:merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants