From 325ca24dce02ef8dfcfd32bf2d9fd97769800bbe Mon Sep 17 00:00:00 2001
From: Rain <rain@oxide.computer>
Date: Tue, 1 Sep 2026 12:43:23 -0700
Subject: [PATCH] [dropshot_endpoint] reuse is_wildcard_path in endpoint
 validation

Make `EndpointMetadata::validate` consistent with `ChannelMetadata::validate` which already uses this helper.

Also fix a typo in the ChannelMetadata doc comment. No functional changes.
---
 dropshot_endpoint/src/metadata.rs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/dropshot_endpoint/src/metadata.rs b/dropshot_endpoint/src/metadata.rs
index 8b469c65b..692d79ef8 100644
--- a/dropshot_endpoint/src/metadata.rs
+++ b/dropshot_endpoint/src/metadata.rs
@@ -104,7 +104,7 @@ impl EndpointMetadata {
             ));
         }
 
-        if path.contains(":.*}") && !self.unpublished {
+        if is_wildcard_path(&path) && !self.unpublished {
             errors.push(Error::new_spanned(
                 attr,
                 format!(
@@ -459,7 +459,7 @@ impl ChannelMetadata {
         get_crate(self._dropshot_crate.as_deref())
     }
 
-    /// Validates metadata, returning a `ValidatedChannelMetadata` is valid.
+    /// Validates metadata, returning a `ValidatedChannelMetadata` if valid.
     ///
     /// Note: the only reason we pass in attr here is to provide a span for
     /// error reporting. As of Rust 1.76, just passing in `attr.span()` produces