You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sentry.apache.org by am...@apache.org on 2019/01/19 01:39:19 UTC

[sentry] branch master updated: SENTRY-2469: Fix bugs in RoleServlet (Hongtaq Zhao reviewed by Arjun Mishra)

This is an automated email from the ASF dual-hosted git repository.

amishra pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/sentry.git


The following commit(s) were added to refs/heads/master by this push:
     new b398a72  SENTRY-2469: Fix bugs in RoleServlet (Hongtaq Zhao reviewed by Arjun Mishra)
b398a72 is described below

commit b398a721fe0051e81b02ea2bee84758cf202845b
Author: amishra <am...@cloudera.com>
AuthorDate: Fri Jan 18 19:37:56 2019 -0600

    SENTRY-2469: Fix bugs in RoleServlet (Hongtaq Zhao reviewed by Arjun Mishra)
    
    Change-Id: I87fbb2e1cd1bb6cc22c93a2069316f9701b6ea26
---
 .../main/java/org/apache/sentry/api/service/thrift/RolesServlet.java    | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/sentry-service/sentry-service-server/src/main/java/org/apache/sentry/api/service/thrift/RolesServlet.java b/sentry-service/sentry-service-server/src/main/java/org/apache/sentry/api/service/thrift/RolesServlet.java
index 9ed2bf3..aea7455 100644
--- a/sentry-service/sentry-service-server/src/main/java/org/apache/sentry/api/service/thrift/RolesServlet.java
+++ b/sentry-service/sentry-service-server/src/main/java/org/apache/sentry/api/service/thrift/RolesServlet.java
@@ -59,6 +59,8 @@ public class RolesServlet extends HttpServlet {
     } catch (Exception e) {
       response.sendError(HttpServletResponse.SC_BAD_REQUEST, e.getMessage());
     }
+    finally {
     out.close();
+    }
   }
 }