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

DomainTools Iris - Investigate Analyzer #572

Merged

Conversation

ChuckWoodraska
Copy link
Contributor

The current DomainTools Analyzers use an older set of APIs. This Analyzer uses the newer DomainTools Iris API for investigating a domain.

@ChuckWoodraska
Copy link
Contributor Author

Added doc PR: TheHive-Project/CortexDocs#54

@EngageCloud
Copy link

@nadouani can you kindly review and approve the above fix? Thanks, much

@EngageCloud
Copy link

@nadouani a kind followup on this. Please lmk if anything is needed from our end.

@jeromeleonard jeromeleonard added this to the 2.6.0 milestone Mar 7, 2020
@jeromeleonard jeromeleonard self-requested a review March 7, 2020 06:39
@jeromeleonard jeromeleonard self-assigned this Mar 7, 2020
@jeromeleonard jeromeleonard added the scope:analyzer Issue is analyzer related label Mar 7, 2020
@jeromeleonard
Copy link
Contributor

Hi @ChuckWoodraska. As we merge DomainToolsIris_Pivot first for last release, I have conflicts while merging this one. Could you please rebase it and check everything is fine in the .py and .html code ?

Regards,

@jeromeleonard
Copy link
Contributor

thx. I think there was a pb on my side. This is working now. merging.

@jeromeleonard jeromeleonard changed the base branch from develop to release/2.6.0 March 10, 2020 08:13
@jeromeleonard jeromeleonard merged commit ee329aa into TheHive-Project:release/2.6.0 Mar 10, 2020
jeromeleonard added a commit that referenced this pull request Mar 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:new-analyzer New analyzer submitted scope:analyzer Issue is analyzer related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants