Add the operation ID to the RequestContext

CHANGELOG.adoc

@@ -18,12 +18,18 @@ https://github.com/oxidecomputer/dropshot/compare/v0.10.1\...HEAD[Full list of c
 === Breaking Changes
 
 * For `ApiEndpoint::register`, the `Err` variant now returns a structured `ApiDescriptionRegisterError` rather than a string.
+* https://github.com/oxidecomputer/dropshot/pull/1087[#1087] The
+`RequestContext` type now contains the `operation_id`, the name of the endpoint
+handler. This is the name of the Rust handler function, if one uses the
+`dropshot::endpoint` macro, and the value of the `operationId` field in the
+generated OpenAPI spec.
 * `TagConfig` field names have changed, for consistency with tag configuration in API traits. The `Deserialize` implementation will still work with the old field names, but the `Serialize` implementation will always produce the new field names.
 ** `endpoint_tag_policy` is now called `policy`.
 ** `tag_definitions` is now called `tags`.
 
 === Other notable changes
 
+
 == 0.10.1 (released 2024-05-15)
 
 https://github.com/oxidecomputer/dropshot/compare/v0.10.0\...v0.10.1[Full list of commits]

dropshot/src/handler.rs

@@ -84,7 +84,8 @@ pub struct RequestContext<Context: ServerContext> {
     pub request_id: String,
     /// logger for this specific request
     pub log: Logger,
-
+    /// The operation ID for the endpoint handler method
+    pub operation_id: String,
     /// basic request information (method, URI, etc.)
     pub request: RequestInfo,
 }

dropshot/src/router.rs

@@ -210,6 +210,7 @@ impl MapValue for VariableValue {
 #[derive(Debug)]
 pub struct RouterLookupResult<Context: ServerContext> {
     pub handler: Arc<dyn RouteHandler<Context>>,
+    pub operation_id: String,
     pub variables: VariableSet,
     pub body_content_type: ApiEndpointBodyContentType,
 }
@@ -482,6 +483,7 @@ impl<Context: ServerContext> HttpRouter<Context> {
             .get(&methodname)
             .map(|handler| RouterLookupResult {
                 handler: Arc::clone(&handler.handler),
+                operation_id: handler.operation_id.clone(),
                 variables,
                 body_content_type: handler.body_content_type.clone(),
             })

dropshot/src/server.rs

@@ -956,6 +956,7 @@ async fn http_request_handle<C: ServerContext>(
         request: RequestInfo::new(&request, remote_addr),
         path_variables: lookup_result.variables,
         body_content_type: lookup_result.body_content_type,
+        operation_id: lookup_result.operation_id,
         request_id: request_id.to_string(),
         log: request_log,
     };

dropshot/src/websocket.rs

@@ -346,6 +346,7 @@ mod tests {
             request: RequestInfo::new(&request, remote_addr),
             path_variables: Default::default(),
             body_content_type: Default::default(),
+            operation_id: "".to_string(),
             request_id: "".to_string(),
             log: log.clone(),
         };