Skip to content

Commit

Permalink
Revert skipped obfuscator tests on PHP (#597)
Browse files Browse the repository at this point in the history
  • Loading branch information
Anilm3 authored Oct 26, 2022
1 parent 55eeab6 commit ac288c2
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 4 deletions.
3 changes: 1 addition & 2 deletions scenarios/appsec/test_traces.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
import pytest

from tests.constants import PYTHON_RELEASE_GA_1_1
from utils import BaseTestCase, context, interfaces, released, rfc, bug
from utils import BaseTestCase, context, interfaces, released, rfc

if context.library == "cpp":
pytestmark = pytest.mark.skip("not relevant")
Expand All @@ -17,7 +17,6 @@
class Test_AppSecObfuscator(BaseTestCase):
"""AppSec obfuscates sensitive data."""

@bug(context.php_appsec == "0.4.0")
def test_appsec_obfuscator_key(self):
"""General obfuscation test of several attacks on several rule addresses."""
# Validate that the AppSec events do not contain the following secret value.
Expand Down
2 changes: 0 additions & 2 deletions tests/appsec/test_traces.py
Original file line number Diff line number Diff line change
Expand Up @@ -167,7 +167,6 @@ def validator(span):
class Test_AppSecObfuscator(BaseTestCase):
"""AppSec obfuscates sensitive data."""

@bug(context.php_appsec == "0.4.0")
def test_appsec_obfuscator_key(self):
"""General obfuscation test of several attacks on several rule addresses."""
# Validate that the AppSec events do not contain the following secret value.
Expand Down Expand Up @@ -216,7 +215,6 @@ def validate_appsec_span_tags(span, appsec_data):
interfaces.library.add_appsec_validation(r, validate_appsec_span_tags)

@missing_feature(library="java")
@bug(context.php_appsec == "0.4.0")
def test_appsec_obfuscator_value(self):
"""Obfuscation test of a matching rule parameter value containing a sensitive keyword."""
# Validate that the AppSec event do not contain the following secret value.
Expand Down

0 comments on commit ac288c2

Please sign in to comment.