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

[Tuning] Connection to Commonly Abused Web Services #3425

Merged
merged 4 commits into from
Apr 2, 2024

Conversation

Samirbous
Copy link
Contributor

extended list of dn.question.name web services.

@Samirbous Samirbous added Rule: Tuning tweaking or tuning an existing rule OS: Windows windows related rules labels Feb 4, 2024
@Samirbous Samirbous self-assigned this Feb 4, 2024
@Samirbous Samirbous requested review from w0rk3r and Aegrah February 5, 2024 11:33
Copy link
Contributor

@Aegrah Aegrah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will this not end up generating too much noise? For example,mail.* will generate many hits given that for example gmail leverages mail.google.com.

Do we wish to alert on all public IP discovery events? If so, do we think that should be part of the "commonly abused web services" rule, or should we create a new discovery rule such as "Public IP Discovery"?

Or would it make sense to turn this one into a BBR, and run a new_terms rule on top of it, for previously unknown dns entries?

@Samirbous
Copy link
Contributor Author

Will this not end up generating too much noise? For example,mail.* will generate many hits given that for example gmail leverages mail.google.com.

removed mail services from rule scope

Do we wish to alert on all public IP discovery events? If so, do we think that should be part of the "commonly abused web services" rule, or should we create a new discovery rule such as "Public IP Discovery"?

removed public IP discovery websvc

Or would it make sense to turn this one into a BBR, and run a new_terms rule on top of it, for previously unknown dns entries?

@Samirbous Samirbous requested a review from Aegrah February 5, 2024 15:04
@Samirbous Samirbous merged commit 6917387 into main Apr 2, 2024
14 checks passed
@Samirbous Samirbous deleted the Samirbous-patch-3 branch April 2, 2024 13:41
protectionsmachine pushed a commit that referenced this pull request Apr 2, 2024
* Update command_and_control_common_webservices.toml

* Update command_and_control_common_webservices.toml

---------

Co-authored-by: Ruben Groenewoud <[email protected]>

(cherry picked from commit 6917387)
protectionsmachine pushed a commit that referenced this pull request Apr 2, 2024
* Update command_and_control_common_webservices.toml

* Update command_and_control_common_webservices.toml

---------

Co-authored-by: Ruben Groenewoud <[email protected]>

(cherry picked from commit 6917387)
protectionsmachine pushed a commit that referenced this pull request Apr 2, 2024
* Update command_and_control_common_webservices.toml

* Update command_and_control_common_webservices.toml

---------

Co-authored-by: Ruben Groenewoud <[email protected]>

(cherry picked from commit 6917387)
protectionsmachine pushed a commit that referenced this pull request Apr 2, 2024
* Update command_and_control_common_webservices.toml

* Update command_and_control_common_webservices.toml

---------

Co-authored-by: Ruben Groenewoud <[email protected]>

(cherry picked from commit 6917387)
protectionsmachine pushed a commit that referenced this pull request Apr 2, 2024
* Update command_and_control_common_webservices.toml

* Update command_and_control_common_webservices.toml

---------

Co-authored-by: Ruben Groenewoud <[email protected]>

(cherry picked from commit 6917387)
protectionsmachine pushed a commit that referenced this pull request Apr 2, 2024
* Update command_and_control_common_webservices.toml

* Update command_and_control_common_webservices.toml

---------

Co-authored-by: Ruben Groenewoud <[email protected]>

(cherry picked from commit 6917387)
protectionsmachine pushed a commit that referenced this pull request Apr 2, 2024
* Update command_and_control_common_webservices.toml

* Update command_and_control_common_webservices.toml

---------

Co-authored-by: Ruben Groenewoud <[email protected]>

(cherry picked from commit 6917387)
protectionsmachine pushed a commit that referenced this pull request Apr 2, 2024
* Update command_and_control_common_webservices.toml

* Update command_and_control_common_webservices.toml

---------

Co-authored-by: Ruben Groenewoud <[email protected]>

(cherry picked from commit 6917387)
protectionsmachine pushed a commit that referenced this pull request Apr 2, 2024
* Update command_and_control_common_webservices.toml

* Update command_and_control_common_webservices.toml

---------

Co-authored-by: Ruben Groenewoud <[email protected]>

(cherry picked from commit 6917387)
protectionsmachine pushed a commit that referenced this pull request Apr 2, 2024
* Update command_and_control_common_webservices.toml

* Update command_and_control_common_webservices.toml

---------

Co-authored-by: Ruben Groenewoud <[email protected]>

(cherry picked from commit 6917387)
protectionsmachine pushed a commit that referenced this pull request Apr 2, 2024
* Update command_and_control_common_webservices.toml

* Update command_and_control_common_webservices.toml

---------

Co-authored-by: Ruben Groenewoud <[email protected]>

(cherry picked from commit 6917387)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport: auto Domain: Endpoint OS: Windows windows related rules Rule: Tuning tweaking or tuning an existing rule
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants