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

cert-request: minor refactors #523

Closed

Conversation

frasertweedale
Copy link
Contributor

@frasertweedale frasertweedale commented Mar 1, 2017

A couple of minor refactors done as part of GSS-API work
(https://pagure.io/freeipa/issue/5011).

Minor refactor to remove the redundant 'principal_type' argument
from 'caacl_check' and associated functions.

Part of: https://pagure.io/freeipa/issue/5011
@tkrizek tkrizek self-assigned this Mar 3, 2017
@tkrizek tkrizek added ack Pull Request approved, can be merged pushed Pull Request has already been pushed labels Mar 3, 2017
@tkrizek
Copy link
Contributor

tkrizek commented Mar 3, 2017

master:

  • 2066a80 Remove redundant principal_type argument
  • 11c9df2 Extract method to map principal to princpal type

@tkrizek tkrizek closed this Mar 3, 2017
@frasertweedale frasertweedale deleted the refactor/5011-cert-request branch March 6, 2017 00:11
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
2 participants