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

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
@frasertweedale frasertweedale force-pushed the fix/8084-kra-with-custom-ca-subj-dn-4.6 branch from 0b260a0 to 94dccb5 Compare October 18, 2019 01:04
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-4.6 branch from 94dccb5 to 7edd015 Compare October 18, 2019 01:23
@frasertweedale frasertweedale added ack Pull Request approved, can be merged pushed Pull Request has already been pushed labels Oct 18, 2019
@frasertweedale
Copy link
Contributor Author

ipa-4-6:

  • 946d96f krainstance: set correct issuer DN in uid=ipakra entry
  • 2fa8c69 upgrade: fix ipakra people entry 'description' attribute
  • 0a0e802 (HEAD) test_integration: add tests for custom CA subject DN

@frasertweedale frasertweedale deleted the fix/8084-kra-with-custom-ca-subj-dn-4.6 branch October 18, 2019 04:59
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