From fe443bedc9e8dbf8e261097039f69fbdd00aa048 Mon Sep 17 00:00:00 2001
From: Anant Vindal <anant.v09@protonmail.com>
Date: Tue, 18 Aug 2026 15:14:22 +0530
Subject: [PATCH] deprecate: GET /role api

---
 src/handlers/http/modal/ingest_server.rs | 2 --
 src/handlers/http/modal/query_server.rs  | 2 --
 src/handlers/http/role.rs                | 2 +-
 3 files changed, 1 insertion(+), 5 deletions(-)

diff --git a/src/handlers/http/modal/ingest_server.rs b/src/handlers/http/modal/ingest_server.rs
index 8d8584339..1910c7d1a 100644
--- a/src/handlers/http/modal/ingest_server.rs
+++ b/src/handlers/http/modal/ingest_server.rs
@@ -146,8 +146,6 @@ impl IngestServer {
     // get the role webscope
     pub fn get_user_role_webscope() -> Scope {
         web::scope("/role")
-            // GET Role List
-            .service(web::resource("").route(web::get().to(role::list).authorize(Action::ListRole)))
             .service(
                 // PUT and GET Default Role
                 web::resource("/default")
diff --git a/src/handlers/http/modal/query_server.rs b/src/handlers/http/modal/query_server.rs
index 6f468be27..19775cf13 100644
--- a/src/handlers/http/modal/query_server.rs
+++ b/src/handlers/http/modal/query_server.rs
@@ -179,8 +179,6 @@ impl QueryServer {
     // get the role webscope
     pub fn get_user_role_webscope() -> Scope {
         web::scope("/role")
-            // GET Role List
-            .service(resource("").route(web::get().to(role::list).authorize(Action::ListRole)))
             .service(
                 // PUT and GET Default Role
                 resource("/default")
diff --git a/src/handlers/http/role.rs b/src/handlers/http/role.rs
index abe8f3dc9..dfeed50bc 100644
--- a/src/handlers/http/role.rs
+++ b/src/handlers/http/role.rs
@@ -107,7 +107,7 @@ pub async fn get(req: HttpRequest, name: web::Path<String>) -> Result<impl Respo
     Ok(web::Json(role))
 }
 
-// Handler for GET /api/v1/role
+// Handler for GET /api/v1/roles
 // Fetch all roles in the system
 pub async fn list(req: HttpRequest) -> Result<impl Responder, RoleError> {
     let tenant_id = get_tenant_id_from_request(&req);