RANGER-5089: Getting 204 instead of 404 status code when invalid serv…

…ice name is passed Signed-off-by: Dineshkumar Yadav <[email protected]>
apache · Jan 6, 2025 · 8a605be · 8a605be
1 parent 6dd1af9
commit 8a605be
Showing 1 changed file with 10 additions and 0 deletions.
diff --git a/security-admin/src/main/java/org/apache/ranger/rest/XUserREST.java b/security-admin/src/main/java/org/apache/ranger/rest/XUserREST.java
@@ -1489,6 +1489,16 @@ public RangerUserStore getSecureRangerUserStoreIfUpdated(@PathParam("serviceName
 		Long    downloadedVersion = null;
 
 		boolean isValid = false;
+		try {
+			isValid = serviceUtil.isValidService(serviceName, request);
+		} catch (WebApplicationException webException) {
+			httpCode = webException.getResponse().getStatus();
+			logMsg = webException.getResponse().getEntity().toString();
+		} catch (Exception e) {
+			httpCode = HttpServletResponse.SC_BAD_REQUEST;
+			logMsg = e.getMessage();
+		}
+
 		try {
 			XXService xService = rangerDaoManager.getXXService().findByName(serviceName);
 			if (xService != null) {