Update Endpoint discovery model (#9277)

What Does This Do
The names of the endpoint attributes that previously used hyphens (e.g., operation-name) have been updated to use underscores (operation_name) to align with the RFC.

Only operation_name and resource_name are now required fields.

The resource_name attribute has been added.

Motivation
The current implementation of endpoint discovery for API Security does not match the latest version of the RFC.

Additional Notes
https://docs.google.com/document/d/1txwuurIiSUWjYX7Xa0let7e49XKW2uhm1djgqjl_gL0/edit?tab=t.0

Author: jandro996

dd-java-agent/instrumentation/spring-webmvc-3.1/src/main/java/datadog/trace/instrumentation/springweb/RequestMappingInfoIterator.java

@@ -69,6 +69,7 @@ private void fetchNext() {
             new Endpoint()
                 .type(Endpoint.Type.REST)
                 .operation(Endpoint.Operation.HTTP_REQUEST)
+                .resource(method + " " + path)
                 .path(path)
                 .method(method)
                 .requestBodyType(requestBody)

dd-java-agent/instrumentation/spring-webmvc-5.3/src/main/java/datadog/trace/instrumentation/springweb/RequestMappingInfoWithPathPatternsIterator.java

@@ -73,6 +73,7 @@ private void fetchNext() {
             new Endpoint()
                 .type(Endpoint.Type.REST)
                 .operation(Endpoint.Operation.HTTP_REQUEST)
+                .resource(method + " " + path)
                 .path(path)
                 .method(method)
                 .requestBodyType(requestBody)

internal-api/src/main/java/datadog/trace/api/telemetry/Endpoint.java

@@ -16,6 +16,7 @@ public class Endpoint {
   private String method;
   private String path;
   private String operation;
+  private String resource;
   private List<String> requestBodyType;
   private List<String> responseBodyType;
   private List<Integer> responseCode;
@@ -67,6 +68,15 @@ public Endpoint operation(final String operation) {
     return this;
   }
 
+  public String getResource() {
+    return resource;
+  }
+
+  public Endpoint resource(final String resource) {
+    this.resource = resource;
+    return this;
+  }
+
   public List<String> getRequestBodyType() {
     return requestBodyType;
   }

telemetry/src/main/java/datadog/telemetry/TelemetryRequestBody.java

@@ -317,18 +317,25 @@ public void beginEndpoints() throws IOException {
 
   public void writeEndpoint(final Endpoint endpoint) throws IOException {
     bodyWriter.beginObject();
-    bodyWriter.name("type").value(endpoint.getType());
-    bodyWriter.name("method").value(endpoint.getMethod());
-    bodyWriter.name("path").value(endpoint.getPath());
-    bodyWriter.name("operation-name").value(endpoint.getOperation());
+    if (endpoint.getType() != null) {
+      bodyWriter.name("type").value(endpoint.getType());
+    }
+    if (endpoint.getMethod() != null) {
+      bodyWriter.name("method").value(endpoint.getMethod());
+    }
+    if (endpoint.getPath() != null) {
+      bodyWriter.name("path").value(endpoint.getPath());
+    }
+    bodyWriter.name("operation_name").value(endpoint.getOperation());
+    bodyWriter.name("resource_name").value(endpoint.getResource());
     if (endpoint.getRequestBodyType() != null) {
       bodyWriter.name("request-body-type").jsonValue(endpoint.getRequestBodyType());
     }
     if (endpoint.getResponseBodyType() != null) {
-      bodyWriter.name("response-body-type").jsonValue(endpoint.getResponseBodyType());
+      bodyWriter.name("response_body_type").jsonValue(endpoint.getResponseBodyType());
     }
     if (endpoint.getResponseCode() != null) {
-      bodyWriter.name("response-code").jsonValue(endpoint.getResponseCode());
+      bodyWriter.name("response_code").jsonValue(endpoint.getResponseCode());
     }
     if (endpoint.getAuthentication() != null) {
       bodyWriter.name("authentication").jsonValue(endpoint.getAuthentication());

telemetry/src/test/groovy/datadog/telemetry/TelemetryServiceSpecification.groovy

@@ -26,7 +26,7 @@ class TelemetryServiceSpecification extends DDSpecification {
   def distribution = new DistributionSeries().namespace("tracers").metric("distro").points([1, 2, 3]).tags(["tag1", "tag2"]).common(false)
   def logMessage = new LogMessage().message("log-message").tags("tag1:tag2").level(LogMessageLevel.DEBUG).stackTrace("stack-trace").tracerTime(32423).count(1)
   def productChange = new ProductChange().productType(ProductChange.ProductType.APPSEC).enabled(true)
-  def endpoint = new Endpoint().first(true).type('REST').method("GET").operation('http.request').path("/test")
+  def endpoint = new Endpoint().first(true).type('REST').method("GET").operation('http.request').resource("GET /test").path("/test")
 
   def 'happy path without data'() {
     setup:

telemetry/src/test/groovy/datadog/telemetry/TestTelemetryRouter.groovy

@@ -273,22 +273,29 @@ class TestTelemetryRouter extends TelemetryRouter {
       def expected = []
       endpoints.each {
         final item = [
-          'type'          : it.type,
-          'method'        : it.method,
-          'path'          : it.path,
-          'operation-name': it.operation
+          'operation_name': it.operation,
+          'resource_name' : it.method + ' ' + it.path,
         ] as Map<String, Object>
+        if (it.type) {
+          item['type'] = it.type
+        }
+        if (it.method) {
+          item['method'] = it.method
+        }
+        if (it.path) {
+          item['path'] = it.path
+        }
         if (it.requestBodyType) {
-          item['request-body-type'] = it.requestBodyType
+          item['request_body_type'] = it.requestBodyType
         }
         if (it.responseBodyType) {
-          item['response-body-type'] = it.responseBodyType
+          item['response_body_type'] = it.responseBodyType
         }
         if (it.authentication) {
           item['authentication'] = it.authentication
         }
         if (it.responseCode) {
-          item['response-code'] = it.responseCode
+          item['response_code'] = it.responseCode
         }
         if (it.metadata) {
           item['metadata'] = it.metadata