Change to deprecatedroute

Signed-off-by: Derek Ho <[email protected]>
opensearch-project · Oct 3, 2024 · fc499c9 · fc499c9
1 parent 18fb634
commit fc499c9
Showing 1 changed file with 16 additions and 21 deletions.
diff --git a/src/main/java/org/opensearch/security/dlic/rest/api/FlushCacheApiAction.java b/src/main/java/org/opensearch/security/dlic/rest/api/FlushCacheApiAction.java
@@ -30,7 +30,6 @@
 import org.opensearch.threadpool.ThreadPool;
 
 import static org.opensearch.security.dlic.rest.api.Responses.internalServerError;
-import static org.opensearch.security.dlic.rest.api.Responses.methodNotImplemented;
 import static org.opensearch.security.dlic.rest.api.Responses.ok;
 import static org.opensearch.security.dlic.rest.support.Utils.addRoutesPrefix;
 
@@ -43,9 +42,21 @@ public class FlushCacheApiAction extends AbstractApiAction {
     private static final List<Route> routes = addRoutesPrefix(
         ImmutableList.of(
             new Route(Method.DELETE, "/cache"),
-            new Route(Method.GET, "/cache"),
-            new Route(Method.PUT, "/cache"),
-            new Route(Method.POST, "/cache")
+            new DeprecatedRoute(
+                Method.GET,
+                "/cache",
+                "GET is not supported for /cache endpoint and will be removed in the next major version."
+            ),
+            new DeprecatedRoute(
+                Method.PUT,
+                "/cache",
+                "PUT is not supported for /cache endpoint and will be removed in the next major version."
+            ),
+            new DeprecatedRoute(
+                Method.POST,
+                "/cache",
+                "POST is not supported for /cache endpoint and will be removed in the next major version."
+            )
         )
     );
 
@@ -95,23 +106,7 @@ public void onFailure(final Exception e) {
 
                     }
                 )
-            )
-            .override(Method.GET, (channel, request, client) -> {
-                deprecationLogger.deprecate("GET", "GET is not supported on this endpoint and will be removed in the next major release.");
-                methodNotImplemented(channel, Method.GET);
-            })
-            .override(Method.POST, (channel, request, client) -> {
-                deprecationLogger.deprecate(
-                    "POST",
-                    "POST is not supported on this endpoint and will be removed in the next major release."
-                );
-                methodNotImplemented(channel, Method.POST);
-            })
-            .override(Method.PUT, (channel, request, client) -> {
-                deprecationLogger.deprecate("PUT", "PUT is not supported on this endpoint and will be removed in the next major release.");
-                methodNotImplemented(channel, Method.PUT);
-            });
-
+            );
     }
 
     @Override