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

[Backport][ipa-4-8] Reset per-indicator Kerberos policy #4058

Closed
wants to merge 1 commit into from

Conversation

tiran
Copy link
Member

@tiran tiran commented Dec 18, 2019

This PR was opened automatically because PR #4057 was pushed to master and backport to ipa-4-8 is required.

When 'ipa krbtpolicy-reset' is called, we need to reset all policy
settings, including per-indicator ones. Per-indicator policy uses
subtyped attributes (foo;bar), the current krbtpolicy-reset code does
not deal with those.

Add support for per-indicator policy reset. It is a bit tricky, as we
need to drop the values to defaults but avoid adding non-per-indicator
variants of the same attributes.

Add test to check that policy has been resetted by observing a new
Kerberos TGT for the user after its policy reset.

Fixes: https://pagure.io/freeipa/issue/8153

Signed-off-by: Alexander Bokovoy <abokovoy@redhat.com>
@tiran tiran added the ack Pull Request approved, can be merged label Dec 18, 2019
@tiran
Copy link
Member Author

tiran commented Dec 18, 2019

PR was ACKed automatically because this is backport of PR #4057. Wait for CI to finish before pushing. In case of questions or problems contact @abbra who is author of the original PR.

@flo-renaud flo-renaud added the pushed Pull Request has already been pushed label Dec 18, 2019
@flo-renaud
Copy link
Contributor

ipa-4-8:

  • a8b52ea Reset per-indicator Kerberos policy

@flo-renaud flo-renaud closed this Dec 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
3 participants