Skip to content

Conversation

@bullerdude
Copy link

@bullerdude bullerdude commented May 10, 2017

New analysers for Cortex:, initial versions:

This was referenced May 10, 2017
@nadouani nadouani self-requested a review July 5, 2017 11:46
@nadouani nadouani added the scope:analyzer Issue is analyzer related label Jul 5, 2017
@nadouani nadouani changed the base branch from master to develop July 5, 2017 12:15
@nadouani
Copy link
Contributor

nadouani commented Jul 5, 2017

Hi guys, I've just tested the IRMA analyzer with an IRMA 1.5.2 VM:

  • Since Cortex 1.1, analyzers require 3 more information: author (author name or org name), license and url.
  • I don't have any experience with IRMA, and I'm wordering if one could enable an authentication mechanism to protect the IRMA APIs, and in this case the analyzer should allow the users to provide at least creads for basic authentication (that was my case, since I've got access to the demo instance demo quarkslab)

Please let know if you want me to add the basic authentication support, otherwise the analyzer is OK, we just need to review the summary() method to produce a taxonomy for mini reports

@BrevilleBro
Copy link

Hi @nadouani, we can update the IRMA analyser when we get some time, unless someone else is happy to take over development of it as it is no longer in use for us.

@nadouani
Copy link
Contributor

nadouani commented Jul 9, 2017

@BrevilleBro that's fine, I'll add what I was talking about. I've reviewed the analyzer and it sounds fine. We're just missing the author information ;)

@3c7
Copy link
Contributor

3c7 commented Dec 14, 2017

Hey @bullerdude @BrevilleBro,
how is the status on this one?

@BrevilleBro
Copy link

Hi @3c7 ,

We are no longer using these and therefore have ceased development of them. They should however, be compatible with the current versions of Cortex, as @nadouani has kindly cleaned them up.

@To-om To-om force-pushed the develop branch 2 times, most recently from d4d772f to fb8f5aa Compare July 29, 2020 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope:analyzer Issue is analyzer related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants