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

Use APIv2 in Onyphe analyers #736

Closed
jeromeleonard opened this issue Apr 3, 2020 · 5 comments · Fixed by #747
Closed

Use APIv2 in Onyphe analyers #736

jeromeleonard opened this issue Apr 3, 2020 · 5 comments · Fixed by #747
Labels
category:feature-request Issue is related to a feature request status:in progress
Milestone

Comments

@jeromeleonard
Copy link
Contributor

As published on https://www.onyphe.io/blog/newsletter-20201-apiv2-new-pricing-and-new-web-search-features/ APIv1 of Onyphe is reaching EoL (July 2020).
@Pierre-Baudry do you have plan to upgrade Onyphe Analyzers and support APIv2 ?

@jeromeleonard jeromeleonard added the category:feature-request Issue is related to a feature request label Apr 3, 2020
@onyphe
Copy link

onyphe commented Apr 16, 2020

Hello,

furthermore, your current implementation relies on Simple API. We suggest you use the Summary API so you can pivot on data based on the following fields:

  • ip
  • domain
  • hostname

And you will get all categories of information in just one Summary API query instead of multiple Simple API queries.

Feel free to contact us at contact at onyphe dot io in case you need help on this subject.

Best regards,

@dadokkio
Copy link
Contributor

I was able to update the code to use the summary and v2 api.
I just have some doubts regarding taxonomies.
If I keep the same approach with summary I obtain 5/10 taxonomies for each analysis.

Eg with some style changes because render was difficult to use
image

I've no idea if all of them are useful or if we can check only threatlist and move the other info in long template. Any tought?

@dadokkio
Copy link
Contributor

dadokkio commented Apr 16, 2020

I added a "verbose_taxonomies" flag in conf that permit you to keep the old approach [1st object] or to just have the threat count [2nd object]

image

@dadokkio
Copy link
Contributor

merging all long templates into one.. this is the result
image

@onyphe
Copy link

onyphe commented Apr 16, 2020

This report view looks really good. You made an awesome work!

@jeromeleonard jeromeleonard added this to the 2.8.0 milestone Apr 24, 2020
@jeromeleonard jeromeleonard linked a pull request May 12, 2020 that will close this issue
@dadokkio dadokkio closed this as completed Jul 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:feature-request Issue is related to a feature request status:in progress
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants