Fix PHP debugger test issues

- Add DD_APPSEC_ENABLED=0 to DebuggerScenario to prevent appsec fields
  from appearing in exception replay snapshot comparisons
- Convert PHP line probes to method probes in set_probes() since PHP
  does not support line-level instrumentation
- Add PHP /debugger/init request in send_rc_apm_tracing_and_probes()
  to ensure RC is processed before the test request fires the probe

Author: bwoebi

tests/debugger/utils.py

@@ -248,16 +248,30 @@ def __get_probe_type(probe_id: str):
                         go_build_dir = {"uds-echo": "echo"}.get(variant, variant)
                         source_file = f"{go_build_dir}/debugger.go"
                     elif language == "php":
-                        source_file = "debugger.php"
-
-                    if uppercase_source_files:
-                        source_file = source_file.upper()
-                    if path_prefix:
-                        source_file = os.path.join(path_prefix, source_file)
-                    if use_backslashes:
-                        source_file = source_file.replace("/", "\\")
+                        # PHP does not support line probes; convert to a method probe.
+                        php_line_to_method = {
+                            "20": "LogProbe",
+                            "71": "expression",
+                        }
+                        lines = probe["where"].get("lines", [])
+                        method = next((php_line_to_method[line] for line in lines if line in php_line_to_method), None)
+                        if method:
+                            probe["where"]["typeName"] = "DebuggerController"
+                            probe["where"]["methodName"] = method
+                            probe["where"]["sourceFile"] = None
+                            probe["where"]["lines"] = []
+                            probe["evaluateAt"] = "EXIT"
+                        else:
+                            source_file = "debugger.php"
 
-                    probe["where"]["sourceFile"] = source_file
+                    if source_file != "":
+                        if uppercase_source_files:
+                            source_file = source_file.upper()
+                        if path_prefix:
+                            source_file = os.path.join(path_prefix, source_file)
+                        if use_backslashes:
+                            source_file = source_file.replace("/", "\\")
+                        probe["where"]["sourceFile"] = source_file
 
                     # Go system-probe requires methodName for line probes to identify the function.
                     # Other languages resolve this from sourceFile+line, but the eBPF-based
@@ -373,6 +387,10 @@ def send_rc_apm_tracing_and_probes(
             )
         )
 
+        # PHP tracer requires a request to /debugger/* to process RC before the test request.
+        if context.library == "php":
+            weblog.get("/debugger/init")
+
     def send_rc_symdb(self, *, reset: bool = True) -> None:
         BaseDebuggerTest._rc_version += 1
         if reset:

utils/_context/_scenarios/debugger.py

@@ -11,6 +11,7 @@ def __init__(self, name: str, doc: str, weblog_env: dict[str, str | None]) -> No
         base_weblog_env: dict[str, str | None] = {
             "DD_REMOTE_CONFIG_ENABLED": "1",
             "DD_REMOTE_CONFIG_POLL_INTERVAL_SECONDS": "1",
+            "DD_APPSEC_ENABLED": "0",
             # added temporarily to troubleshoot debugger test flakiness
             "JAVA_OPTS": " -Ddatadog.slf4j.simpleLogger.log.com.datadog.debugger=debug ",
         }