From 06a385178b6b5ac06ab7c46eee164f19212f2f0d Mon Sep 17 00:00:00 2001
From: "Andrew J. Stone" <andrew.j.stone.1@gmail.com>
Date: Tue, 19 Dec 2023 21:59:22 +0000
Subject: [PATCH] review fixes

---
 nexus/src/external_api/http_entrypoints.rs | 9 ++++++++-
 openapi/nexus.json                         | 7 -------
 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/nexus/src/external_api/http_entrypoints.rs b/nexus/src/external_api/http_entrypoints.rs
index 079a94a640..3e38558760 100644
--- a/nexus/src/external_api/http_entrypoints.rs
+++ b/nexus/src/external_api/http_entrypoints.rs
@@ -4659,9 +4659,16 @@ async fn rack_view(
 }]
 async fn sled_list_uninitialized(
     rqctx: RequestContext<Arc<ServerContext>>,
-    _query_params: Query<PaginatedById>,
+    query: Query<PaginationParams<EmptyScanParams, String>>,
 ) -> Result<HttpResponseOk<ResultsPage<UninitializedSled>>, HttpError> {
     let apictx = rqctx.context();
+    // We don't actually support real pagination
+    let pag_params = query.into_inner();
+    if let dropshot::WhichPage::Next(last_seen) = &pag_params.page {
+        return Err(
+            Error::invalid_value(last_seen.clone(), "bad page token").into()
+        );
+    }
     let handler = async {
         let nexus = &apictx.nexus;
         let opctx = crate::context::op_context_for_external_api(&rqctx).await?;
diff --git a/openapi/nexus.json b/openapi/nexus.json
index aa0cbd4a79..4dcfda7f78 100644
--- a/openapi/nexus.json
+++ b/openapi/nexus.json
@@ -4042,13 +4042,6 @@
               "nullable": true,
               "type": "string"
             }
-          },
-          {
-            "in": "query",
-            "name": "sort_by",
-            "schema": {
-              "$ref": "#/components/schemas/IdSortMode"
-            }
           }
         ],
         "responses": {