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.

Sign up for GitHub

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

Jump to bottom

"VirusTotal_Scan" analyzer is not checking for TLP #2

Closed
jeromeleonard opened this issue Feb 14, 2017 · 1 comment
Closed

"VirusTotal_Scan" analyzer is not checking for TLP #2

jeromeleonard opened this issue Feb 14, 2017 · 1 comment
Assignees
@jeromeleonard
Labels
scope:analyzer Issue is analyzer related

Comments

@jeromeleonard
Copy link
Contributor

Setting TLP:AMBER or up is not avoiding sending files to VT for scan.
there is no max_tlp in the Virustotal scan analyser configuration, and tlp is not checked in the main program
@jeromeleonard jeromeleonard changed the title VirusTotal_Scan analyzer is not checking for TLP "VirusTotal_Scan" analyzer is not checking for TLP Feb 14, 2017
@jeromeleonard jeromeleonard self-assigned this Feb 14, 2017
@jeromeleonard jeromeleonard added the scope:analyzer Issue is analyzer related label Feb 14, 2017
jeromeleonard added a commit that referenced this issue Feb 14, 2017
@jeromeleonard
#2 set check_tlp to True in analyzer
25dee1f
jeromeleonard added a commit that referenced this issue Feb 14, 2017
@jeromeleonard
#2 add max_tlp in config
4a2476a
jeromeleonard added a commit that referenced this issue Feb 14, 2017
@jeromeleonard
#2 move check_tlp in JSON config file of VirusTotal_Scan
7bb44d1
@ghost
Copy link

ghost commented Feb 14, 2017

Hi !

Little error in the following file : https://github.com/CERT-BDF/Cortex-Analyzers/blob/master/analyzers/Fortiguard/Fortiguard_URLCategory.json

jeromeleonard added a commit that referenced this issue Feb 14, 2017
@jeromeleonard
#2 fix Fortiguard_URLCategory.json
aea29ec
To-om pushed a commit that referenced this issue Apr 17, 2019
@saadkadhi
Merge pull request #2 from SHSauler/patch-1
89253d6 
Fix: cleanup of HTTPS FAQ
dadokkio pushed a commit that referenced this issue Feb 3, 2021
@markus-nclose
Merge pull request #2 from Nclose-ZA/greynoise_analyzer_v3
a931f79 
Greynoise analyzer v3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jeromeleonard jeromeleonard

Labels
scope:analyzer Issue is analyzer related
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jeromeleonard