cb5b89f83e
* adds scripting to include deprecated rule stubs in package * remove deprecated manifest from package * adds 9.4 gate * bump version * fix merge conflict * test * revert commit hash * adds deprecated_reason logic from comment * fix lint error * fix lint error * fix formatting * test * revert commit hash * Update detection_rules/packaging.py Co-authored-by: Mika Ayenson, PhD <Mikaayenson@users.noreply.github.com> --------- Co-authored-by: Mika Ayenson, PhD <Mikaayenson@users.noreply.github.com>
202 lines
8.3 KiB
Python
202 lines
8.3 KiB
Python
# Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
|
|
# or more contributor license agreements. Licensed under the Elastic License
|
|
# 2.0; you may not use this file except in compliance with the Elastic License
|
|
# 2.0.
|
|
|
|
"""Test that the packages are built correctly."""
|
|
|
|
import unittest
|
|
import uuid
|
|
from pathlib import Path
|
|
|
|
from marshmallow import ValidationError
|
|
from semver import Version
|
|
|
|
from detection_rules import rule_loader
|
|
from detection_rules.packaging import (
|
|
PACKAGE_FILE,
|
|
Package,
|
|
build_deprecated_rule_asset,
|
|
)
|
|
from detection_rules.rule_loader import RuleCollection
|
|
from detection_rules.schemas import definitions
|
|
from detection_rules.schemas.registry_package import RegistryPackageManifestV1, RegistryPackageManifestV3
|
|
from tests.base import BaseRuleTest
|
|
|
|
package_configs = Package.load_configs()
|
|
|
|
|
|
class TestPackages(BaseRuleTest):
|
|
"""Test package building and saving."""
|
|
|
|
@staticmethod
|
|
def get_test_rule(version=1, count=1):
|
|
def get_rule_contents():
|
|
return {
|
|
"author": ["Elastic"],
|
|
"description": "test description",
|
|
"language": "kuery",
|
|
"license": "Elastic License v2",
|
|
"name": "test rule",
|
|
"query": "process.name:test.query",
|
|
"risk_score": 21,
|
|
"rule_id": str(uuid.uuid4()),
|
|
"severity": "low",
|
|
"type": "query",
|
|
}
|
|
|
|
rules = [rule_loader.TOMLRule("test.toml", get_rule_contents()) for i in range(count)]
|
|
version_info = {
|
|
rule.id: {"rule_name": rule.name, "sha256": rule.contents.get_hash(), "version": version} for rule in rules
|
|
}
|
|
|
|
return rules, version_info
|
|
|
|
def test_package_loader_production_config(self):
|
|
"""Test that packages are loading correctly."""
|
|
|
|
@unittest.skipIf(rule_loader.RULES_CONFIG.bypass_version_lock, "Version lock bypassed")
|
|
def test_package_loader_default_configs(self):
|
|
"""Test configs in detection_rules/etc/packages.yaml."""
|
|
Package.from_config(rule_collection=self.rc, config=package_configs)
|
|
|
|
@unittest.skipIf(rule_loader.RULES_CONFIG.bypass_version_lock, "Version lock bypassed")
|
|
def test_package_summary(self):
|
|
"""Test the generation of the package summary."""
|
|
rules = self.rc
|
|
package = Package(rules, "test-package")
|
|
package.generate_summary_and_changelog(package.changed_ids, package.new_ids, package.removed_ids)
|
|
|
|
@unittest.skipIf(rule_loader.RULES_CONFIG.bypass_version_lock, "Version lock bypassed")
|
|
def test_rule_versioning(self):
|
|
"""Test that all rules are properly versioned and tracked"""
|
|
self.maxDiff = None
|
|
rules = self.rc
|
|
original_hashes = []
|
|
|
|
# test that no rules have versions defined
|
|
for rule in rules:
|
|
self.assertGreaterEqual(rule.contents.autobumped_version, 1, "{} - {}: version is not being set in package")
|
|
original_hashes.append(rule.contents.get_hash())
|
|
|
|
package = Package(rules, "test-package")
|
|
|
|
# test that all rules have versions defined
|
|
for rule in package.rules:
|
|
self.assertGreaterEqual(rule.contents.autobumped_version, 1, "{} - {}: version is not being set in package")
|
|
|
|
# test that rules validate with version
|
|
|
|
post_bump_hashes = [rule.contents.get_hash() for rule in package.rules]
|
|
|
|
# test that no hashes changed as a result of the version bumps
|
|
self.assertListEqual(original_hashes, post_bump_hashes, "Version bumping modified the hash of a rule")
|
|
|
|
|
|
class TestRegistryPackage(unittest.TestCase):
|
|
"""Test the OOB registry package."""
|
|
|
|
@classmethod
|
|
def setUpClass(cls) -> None:
|
|
assert "registry_data" in package_configs, f"Missing registry_data in {PACKAGE_FILE}"
|
|
cls.registry_config = package_configs["registry_data"]
|
|
stack_version = Version.parse(
|
|
cls.registry_config["conditions"]["kibana.version"].strip("^"), optional_minor_and_patch=True
|
|
)
|
|
if stack_version >= Version.parse("8.12.0"):
|
|
RegistryPackageManifestV3.from_dict(cls.registry_config)
|
|
else:
|
|
RegistryPackageManifestV1.from_dict(cls.registry_config)
|
|
|
|
def test_registry_package_config(self):
|
|
"""Test that the registry package is validating properly."""
|
|
registry_config = self.registry_config.copy()
|
|
registry_config["version"] += "7.1.1."
|
|
|
|
with self.assertRaises(ValidationError):
|
|
RegistryPackageManifestV1.from_dict(registry_config)
|
|
|
|
|
|
class TestDeprecatedRuleAsset(unittest.TestCase):
|
|
"""Test deprecated rule stub asset building and version gate."""
|
|
|
|
def test_build_deprecated_rule_asset_structure(self):
|
|
"""build_deprecated_rule_asset returns a dict with id, type, and attributes."""
|
|
asset = build_deprecated_rule_asset(
|
|
rule_id="aaaaaaaa-bbbb-4ccc-dddd-eeeeeeeeeeee",
|
|
rule_name="Deprecated Rule Name",
|
|
deprecated_version=3,
|
|
)
|
|
self.assertIsInstance(asset, dict)
|
|
self.assertEqual(asset["id"], "aaaaaaaa-bbbb-4ccc-dddd-eeeeeeeeeeee_3")
|
|
self.assertEqual(asset["type"], definitions.SAVED_OBJECT_TYPE)
|
|
self.assertIn("attributes", asset)
|
|
self.assertEqual(asset["attributes"]["rule_id"], "aaaaaaaa-bbbb-4ccc-dddd-eeeeeeeeeeee")
|
|
self.assertEqual(asset["attributes"]["version"], 3)
|
|
self.assertEqual(asset["attributes"]["name"], "Deprecated Rule Name")
|
|
self.assertIs(asset["attributes"]["deprecated"], True)
|
|
|
|
def test_build_deprecated_rule_asset_serializable(self):
|
|
"""Asset is JSON-serializable and matches expected shape for package."""
|
|
import json
|
|
|
|
asset = build_deprecated_rule_asset(
|
|
rule_id="00000000-0000-4000-8000-000000000001",
|
|
rule_name="Test",
|
|
deprecated_version=1,
|
|
)
|
|
# Should not raise
|
|
json_str = json.dumps(asset, indent=4, sort_keys=True)
|
|
loaded = json.loads(json_str)
|
|
self.assertEqual(loaded["attributes"]["deprecated"], True)
|
|
self.assertEqual(loaded["type"], "security-rule")
|
|
|
|
def test_build_deprecated_rule_asset_deprecated_reason_only_on_94(self):
|
|
"""deprecated_reason is only added when stack_version is 9.4+ (Kibana feature)."""
|
|
reason = "Replaced by rule X"
|
|
# With stack_version None or < 9.4, deprecated_reason must not appear
|
|
for stack_ver in (None, Version(9, 3, 0)):
|
|
with self.subTest(stack_version=stack_ver):
|
|
asset = build_deprecated_rule_asset(
|
|
rule_id="aaaaaaaa-bbbb-4ccc-dddd-eeeeeeeeeeee",
|
|
rule_name="Deprecated Rule",
|
|
deprecated_version=2,
|
|
deprecated_reason=reason,
|
|
stack_version=stack_ver,
|
|
)
|
|
self.assertNotIn("deprecated_reason", asset["attributes"])
|
|
# With stack_version >= 9.4 it appears
|
|
asset = build_deprecated_rule_asset(
|
|
rule_id="aaaaaaaa-bbbb-4ccc-dddd-eeeeeeeeeeee",
|
|
rule_name="Deprecated Rule",
|
|
deprecated_version=2,
|
|
deprecated_reason=reason,
|
|
stack_version=Version(9, 4, 0),
|
|
)
|
|
self.assertEqual(asset["attributes"]["deprecated_reason"], reason)
|
|
|
|
@unittest.skipIf(rule_loader.RULES_CONFIG.bypass_version_lock, "Version lock bypassed")
|
|
def test_deprecated_rule_load_dict_preserves_deprecated_reason(self):
|
|
"""Loading a deprecated rule dict with deprecated_reason in [metadata] preserves it."""
|
|
rule_id = "aaaaaaaa-bbbb-4ccc-dddd-eeeeeeeeeeee"
|
|
reason = "Replaced by rule X"
|
|
obj = {
|
|
"metadata": {
|
|
"creation_date": "2020/01/01",
|
|
"deprecation_date": "2024/01/01",
|
|
"updated_date": "2024/01/01",
|
|
"maturity": "deprecated",
|
|
"deprecated_reason": reason,
|
|
},
|
|
"rule": {
|
|
"rule_id": rule_id,
|
|
"name": "Test Deprecated Rule",
|
|
"description": "Minimal deprecated rule for test",
|
|
},
|
|
}
|
|
collection = RuleCollection()
|
|
path = Path("rules/_deprecated/test_deprecated_reason.toml")
|
|
rule = collection.load_dict(obj, path=path)
|
|
self.assertIn("deprecated_reason", rule.contents.metadata)
|
|
self.assertEqual(rule.contents.metadata["deprecated_reason"], reason)
|