Skip to content

Conversation

@darses
Copy link
Contributor

@darses darses commented Jan 2, 2026

Summary of the Pull Request

Add new OAST domains and set modifier to endswith.

Changelog

update: DNS Query to External Service Interaction Domains - Add domains digimg.store, instances.httpworkbench.com, odiss.eu.
update: DNS Query to External Service Interaction Domains - Change modifier from contains to endswith

Example Log Event

Fixed Issues

SigmaHQ Rule Creation Conventions

  • If your PR adds new rules, please consider following and applying these conventions

@github-actions github-actions bot added Rules Review Needed The PR requires review labels Jan 2, 2026
@darses darses changed the title Add new OAST domains update: DNS Query to External Service Interaction Domains - Add domains change modifier Jan 2, 2026
@swachchhanda000 swachchhanda000 added the Author Input Required changes the require information from original author of the rules label Jan 5, 2026
@swachchhanda000 swachchhanda000 removed the Author Input Required changes the require information from original author of the rules label Jan 5, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the DNS Query to External Service Interaction Domains detection rule by adding three new OAST (Out-of-Application Security Testing) domains and changing the matching modifier from contains to endswith for more precise domain matching.

Key Changes:

  • Changed the query modifier from contains to endswith to prevent false positives from substring matches
  • Added three new OAST domains: .digimg.store, .instances.httpworkbench.com, and .odiss.eu
  • Added clarifying comments to identify the service providers for various OAST domains

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Review Needed The PR requires review Rules

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants