-
Notifications
You must be signed in to change notification settings - Fork 7
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
Create sophos-endpoint-security #96
Conversation
Initial barebones build. Will updated further once integration goes into beta
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.
some updates needed, great start!
rule-packs/sophos-endpoint-security
Outdated
[ | ||
{ | ||
"name": "sophos-misconfigured-endpoints-1", | ||
"description": "This will query for endpoints that do not have sophos edr installed.", |
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.
'that do not have the Sophos EDR Agent installed."
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.
Adjusting
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.
corrections made per the comments
rule-packs/sophos-endpoint-security
Outdated
[ | ||
{ | ||
"name": "sophos-misconfigured-endpoints-1", | ||
"description": "This will query for endpoints that do not have sophos edr installed.", |
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.
Adjusting
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.
adjusting the index.js
change syntax issue
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.
change syntax issue
…y.json testing possible build fix
" -> ' added commas before alertLevel removed comma on 145
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.
a few syntax updates.
Initial barebones build. Will updated further once integration goes into beta
QA Checklist
Alerts Rule Packs