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

New Responder KnowBe4 #549

Merged
merged 9 commits into from Jan 8, 2020
Merged

Conversation

@arnydo
Copy link
Contributor

arnydo commented Oct 21, 2019

Addresses #548

arnydo and others added 9 commits Jan 8, 2019
Pull in latest release
arnydo
@arnydo

This comment has been minimized.

Copy link
Contributor Author

arnydo commented Oct 23, 2019

Sample event
image

@nadouani nadouani added this to the 2.4.0 milestone Jan 8, 2020
@nadouani nadouani removed the responder label Jan 8, 2020
@nadouani

This comment has been minimized.

Copy link
Contributor

nadouani commented Jan 8, 2020

Hello @arnydo Thanks for the PR.

@nadouani

This comment has been minimized.

Copy link
Contributor

nadouani commented Jan 8, 2020

@nadouani nadouani self-requested a review Jan 8, 2020
@nadouani nadouani added needs-docs and removed pr-submitted labels Jan 8, 2020
@arnydo

This comment has been minimized.

Copy link
Contributor Author

arnydo commented Jan 8, 2020

Of course. Added PR here TheHive-Project/CortexDocs#53

@nadouani nadouani removed the needs-docs label Jan 8, 2020
@nadouani nadouani merged commit a155548 into TheHive-Project:develop Jan 8, 2020
1 check failed
1 check failed
continuous-integration/drone/pr Build is failing
Details
nadouani added a commit that referenced this pull request Jan 8, 2020
@nadouani

This comment has been minimized.

Copy link
Contributor

nadouani commented Jan 8, 2020

I've renamed hive_url to thehive_url :)

@nadouani

This comment has been minimized.

Copy link
Contributor

nadouani commented Jan 8, 2020

It looks like the KnowBe4 folder is duplicate in analyzers and responders folder, is this expected? I don't think so :)

nadouani added a commit that referenced this pull request Jan 8, 2020
@arnydo

This comment has been minimized.

Copy link
Contributor Author

arnydo commented Jan 8, 2020

Woops. No, it should have only been in responders...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.