Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Auth Lib - Delete principal function #6029

Closed
alansemenov opened this issue Feb 6, 2018 · 3 comments
Closed

Auth Lib - Delete principal function #6029

alansemenov opened this issue Feb 6, 2018 · 3 comments
Assignees
@GlennRicaud
Labels
Feature
Milestone
6.14.0

Comments

@alansemenov
Copy link
Member

No description provided.

@alansemenov alansemenov added the Bug label Feb 6, 2018
@alansemenov alansemenov added this to the 6.14.0 milestone Feb 6, 2018
@alansemenov alansemenov mentioned this issue Feb 6, 2018
Closed
@sigdestad
Copy link
Member

This needs to be handled in the very core of the auth API (applies to system userstore only).

@GlennRicaud
Copy link
Contributor

GlennRicaud commented Feb 8, 2018 •
edited

There is no deletePrincipal in the auth library.

  • I will modify this issue to a feature 'Auth Lib - Delete principal function'
    • This function will additionally have a check for user:system:su and user:system:anonymous
  • I will create an XP Apps issue to use this function instead

@GlennRicaud GlennRicaud changed the title Don't allow deleting SU and Anonymous users Auth Lib - Delete principal Feb 8, 2018
@GlennRicaud GlennRicaud changed the title Auth Lib - Delete principal Auth Lib - Delete principal function Feb 8, 2018
@GlennRicaud GlennRicaud added Feature and removed Bug labels Feb 8, 2018
@sigdestad
Copy link
Member

You are right - I do not want to know it :-)

@GlennRicaud GlennRicaud mentioned this issue Feb 8, 2018
Closed
GlennRicaud pushed a commit that referenced this issue Feb 8, 2018
Auth Lib - Delete principal function (#6029)
0be6d86
GlennRicaud pushed a commit that referenced this issue Feb 9, 2018
Auth Lib - Delete principal function (#6029)
93a08d4
GlennRicaud pushed a commit that referenced this issue Feb 9, 2018
Auth Lib - Delete principal function (#6029)
d070944
GlennRicaud pushed a commit that referenced this issue Feb 9, 2018
Auth Lib - Delete principal function (#6029)
9ebf91c
aro pushed a commit that referenced this issue Feb 9, 2018
Merge pull request #6035 from enonic/issue-6029
5f02029 
 Auth Lib - Delete principal function (#6029)
@jsi jsi closed this as completed Feb 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@GlennRicaud GlennRicaud

Labels
Feature
Projects
None yet
Milestone
6.14.0
Development

No branches or pull requests
4 participants
@jsi @sigdestad @alansemenov @GlennRicaud