[freeipa PR#5915][opened] Clean up the PKI securitydomain when removing a server
by rcritten
URL: https://github.com/freeipa/freeipa/pull/5915
Author: rcritten
Title: #5915: Clean up the PKI securitydomain when removing a server
Action: opened
PR body:
"""
PKI has its own internal knowledge of servers and services
in its securitydomain. This has not been cleaned up in the
past but is becoming more of an issue as PKI now relies on its
securitydomain for more things, and it has a healthcheck that
reports inconsistencies.
Removing entries is straightforward using the PKI REST API.
In order to operate on the API access is needed. There was an
unused Security Domain Administrators group that I've added to
the resourceACLS we created for managing the securitydomain.
The ipara user is added as a member of this group. The REST
API binds to the CA using the IPA RA certificate.
Related commits are b3c2197b7e4ed18a7febe3efa6396c2272ebccca
and ba4df6449aaa0843ab43a1a2b3cb1df8bb022c24.
These resourceACLS were originally created as a backwards
compatibility mechanism for dogtag v9 and later only created when a
replica was installed purportedly to save a restart. I don't see
any reason to not have these defined. They are apparently needed due
to the PKI database upgrade issues.
In any case if the purpose was to suppress these ACLS it failed
because as soon as a replica with a CA was installed they were as
well, and we need this ACL in order to manage the securitydomain.
https://pagure.io/freeipa/issue/8930
Signed-off-by: Rob Crittenden <rcritten(a)redhat.com>
"""
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/5915/head:pr5915
git checkout pr5915
2 years, 1 month
[freeipa PR#5932][opened] Parse getStatus as JSON not XML
by ckelleyRH
URL: https://github.com/freeipa/freeipa/pull/5932
Author: ckelleyRH
Title: #5932: Parse getStatus as JSON not XML
Action: opened
PR body:
"""
On `dogtagpki/pki` master XML is being replaced by JSON, getStatus will return JSON in PKI 11.0+.
The PR for `dogtagpki/pki` that makes this change necessary is: https://github.com/dogtagpki/pki/pull/3674
To test, I forked the nightly copr build of `freeipa` and added in this patch. I then ran the `dogtagpki/pki` CI against my copr fork instead of the nightly freeipa copr. `dogtagpki/pki` CI fails against the nightly `freeipa` build, but succeeds with my patched build.
This is my first PR to this repo (hello everyone!) so if I need to do anything in addition, like bump a version or document that there is a breaking API change, then I would be grateful if someone could point me in the right direction!
"""
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/5932/head:pr5932
git checkout pr5932
2 years, 1 month