-
Notifications
You must be signed in to change notification settings - Fork 321
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
Cybereason #306
Cybereason #306
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Test should pass once #291 merged in. Will do that and confirm.
@FlorianBracq one additional thing, could we ask that you please add a short documentation page under docs/source/data_acquisiton. Just something very short to show users how to access the use the provider. |
Thanks for the docs @FlorianBracq - looks like you have an issue with the Cybereason tests. It looks like the mocked data in the test has this format: |
@petebryan: that was a PEBKAC ;) |
Answers #305 but requires #285 and #291 to work as expected.