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.

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

sudo: allow use of default domain suffix (sssd-1-13) #595

Closed
wants to merge 1 commit into from

Conversation

Projects
None yet
2 participants
@pbrezina
Copy link
Member

commented Jun 11, 2018

This patch is based on post 1.13 work. The purpose is to allow combination of fully qualified names and default domain suffix when user log in with a short name.

Similarly to later version, we add #uid value to sudoUser attribute.

@jhrozek jhrozek changed the title sudo: allow use of default domain suffix sudo: allow use of default domain suffix (sssd-1-13) Jun 11, 2018

@jhrozek

This comment has been minimized.

Copy link
Contributor

commented Jun 20, 2018

Thank you, the patch looks good and seems to work fine.

@jhrozek

This comment has been minimized.

Copy link
Contributor

commented Jun 20, 2018

I started some downstream tests before giving the accepted label.

@jhrozek jhrozek self-assigned this Jul 10, 2018

@jhrozek jhrozek added the Accepted label Jul 10, 2018

@jhrozek

This comment has been minimized.

Copy link
Contributor

commented Jul 10, 2018

The tests passed, I just forgot to push the patch to sssd-1-13..

@jhrozek

This comment has been minimized.

Copy link
Contributor

commented Jul 10, 2018

@jhrozek jhrozek closed this Jul 10, 2018

@jhrozek jhrozek added the Pushed label Jul 10, 2018

@pbrezina pbrezina deleted the pbrezina:sudo branch Aug 16, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.