From dev-return-893-apmail-ranger-dev-archive=ranger.apache.org@ranger.incubator.apache.org Tue Jan 6 23:41:54 2015 Return-Path: X-Original-To: apmail-ranger-dev-archive@www.apache.org Delivered-To: apmail-ranger-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id CA52017FF1 for ; Tue, 6 Jan 2015 23:41:54 +0000 (UTC) Received: (qmail 65569 invoked by uid 500); 6 Jan 2015 23:41:55 -0000 Delivered-To: apmail-ranger-dev-archive@ranger.apache.org Received: (qmail 65532 invoked by uid 500); 6 Jan 2015 23:41:55 -0000 Mailing-List: contact dev-help@ranger.incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@ranger.incubator.apache.org Delivered-To: mailing list dev@ranger.incubator.apache.org Received: (qmail 65521 invoked by uid 99); 6 Jan 2015 23:41:55 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 06 Jan 2015 23:41:55 +0000 X-ASF-Spam-Status: No, hits=-2000.0 required=5.0 tests=ALL_TRUSTED,T_RP_MATCHES_RCVD X-Spam-Check-By: apache.org Received: from [140.211.11.3] (HELO mail.apache.org) (140.211.11.3) by apache.org (qpsmtpd/0.29) with SMTP; Tue, 06 Jan 2015 23:41:54 +0000 Received: (qmail 64927 invoked by uid 99); 6 Jan 2015 23:41:34 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 06 Jan 2015 23:41:34 +0000 Date: Tue, 6 Jan 2015 23:41:34 +0000 (UTC) From: "Alok Lal (JIRA)" To: dev@ranger.incubator.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Commented] (RANGER-205) Delete rest api of User not deleting user completely from system MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 X-Virus-Checked: Checked by ClamAV on apache.org [ https://issues.apache.org/jira/browse/RANGER-205?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14266982#comment-14266982 ] Alok Lal commented on RANGER-205: --------------------------------- {quote} Soft delete the user/group if they are internal(i.e. created from UI). {quote} Agreed. {quote} Give the error message if someone try to delete (using rest apis) external user/group . {quote} Do we want to have exception for REST api, i.e. do you want to allow this via the UI? Should API call succeed, i.e. get back 200? I feel it should be disallowed via API or UI. REST call should return 500. {quote} Give the error message if there is some policy defined for user/group that is to be deleted. {quote} Not sure I understand this. Can you elaborate the use case a bit please? Fact that a policy exists should not prevent soft delete of an internal user. Such a policy does no harm other than perhaps to slow things down and lead to some clutter. Such policies should simply get disabled as a side effect and user soft-delete be allowed. I would say we return 200 still. > Delete rest api of User not deleting user completely from system > ---------------------------------------------------------------- > > Key: RANGER-205 > URL: https://issues.apache.org/jira/browse/RANGER-205 > Project: Ranger > Issue Type: Bug > Affects Versions: 0.4.0 > Reporter: Hanish Bansal > > Delete rest api of user is removing entry from x_user table of database and not from x_portal_user table so user get invisible from UI and api return success status. > Due to which if a user is updated to have username/email-id of deleted user, it will error message that this username/email-id already exists. > Also if a new user is created with user-name of deleted user, user get successfully created but his/her details are mapped with deleted user. > Rest apis used are: > {quote} > DELETE http://:6080/service/xusers/users/userName/ > DELETE http://:6080/service/xusers/users/ > {quote} > Expected result: > Apis should remove entry from both tables x_user and x_portal_user. -- This message was sent by Atlassian JIRA (v6.3.4#6332)