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] krainstance: set correct issuer DN in uid=ipakra entry #3804

Conversation

frasertweedale
Copy link
Contributor

Manual backport of #3764

If IPA CA has custom subject DN (not "CN=Certificate
Authority,{subject_base}"), the uid=ipakra people entry gets an
incorrect 'description' attribute.  The issuer DN in the
'description' attribute is based on the aforementioned pattern,
instead of the actual IPA CA subject DN.

Update KRAInstance.configure_instance() to require the CA subject DN
argument.  Update ipaserver.install.kra.install() to pass the CA
subject DN.

Fixes: https://pagure.io/freeipa/issue/8084
Add an upgrade script to detect when ipakra people entry has
incorrect 'description' attribute and fix it.

Part of: https://pagure.io/freeipa/issue/8084
Define integration test for custom CA subject DN and subject base
scenarios.  Add to nightly CI runs.

Part of: https://pagure.io/freeipa/issue/8084
@frasertweedale frasertweedale force-pushed the fix/8084-kra-with-custom-ca-subj-dn branch from 8456c20 to 78c6d3b Compare October 17, 2019 07:50
@frasertweedale
Copy link
Contributor Author

@frasertweedale frasertweedale force-pushed the fix/8084-kra-with-custom-ca-subj-dn branch from 78c6d3b to 0cadebf Compare October 17, 2019 10:25
@frasertweedale
Copy link
Contributor Author

CI is green, adding the ack.

@frasertweedale frasertweedale added ack Pull Request approved, can be merged pushed Pull Request has already been pushed labels Oct 17, 2019
@frasertweedale
Copy link
Contributor Author

ipa-4-8:

  • 09c6db7 krainstance: set correct issuer DN in uid=ipakra entry
  • 23f4e00 upgrade: fix ipakra people entry 'description' attribute
  • 5d68d04 (HEAD) test_integration: add tests for custom CA subject DN

@frasertweedale frasertweedale deleted the fix/8084-kra-with-custom-ca-subj-dn branch October 18, 2019 01:52
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
1 participant