This is an automated email from the ASF dual-hosted git repository.
dineshkumar pushed a commit to branch ranger-2.6
in repository https://gitbox.apache.org/repos/asf/ranger.git
The following commit(s) were added to refs/heads/ranger-2.6 by this push:
new a63299768 RANGER-5089: Getting 204 instead of 404 status code when
invalid service name is passed
a63299768 is described below
commit a632997681b85ab635d3b471d1a0002c47466a17
Author: Rakesh Gupta <[email protected]>
AuthorDate: Fri Jan 3 18:00:16 2025 +0530
RANGER-5089: Getting 204 instead of 404 status code when invalid service
name is passed
Signed-off-by: Dineshkumar Yadav <[email protected]>
---
.../src/main/java/org/apache/ranger/rest/XUserREST.java | 10 ++++++++++
1 file changed, 10 insertions(+)
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
index 9f7e79f7f..020cf6184 100755
--- 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 class XUserREST {
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) {
