The patch appears to do pretty much what we discussed., so with the
comments below, ACK.
Couple of comments:
1. default_tks_admin_hostname etc. appear to be defined as global
variables, when they are in fact only used in main. Lets define them
there where they are used. I know we have a huge list of globals in
pkisilent, but thats just bad practice.
2. I'm a little confused as to why the "empty" value is being used. It
seems to the equivalent state as "". I'm assuming its because you added
this value to the templates, and that this is more intuitive than
setting a parameter value to "" or not setting it at all?
Ade
On Thu, 2013-08-01 at 20:41 -0700, Matthew Harmsen wrote:
Please review the attached patch for the following RHCS 8.1 bug:
* Bugzilla Bug #979559 - Parameter --ca_domain_url should be
optional
This bug addresses the problem of attempting to configure an instance
using a version of pki-silent which contains the new code with
expanded parameters while using an old template from a previous
version of pki-silent which did not contain these parameters.
Tested by installing and successfully configuring a CA, KRA, TKS, and
TPS using legacy templates as well as successfully configuring a CA,
KRA, TKS, and TPS using the new templates.
The attached code was revised to exclude the DRM/TKS agent
hostname:port and provide more verbose help messages regarding the
optional new URL parameters.
For readability sake, the new URL help messages follow:
# pkisilent ConfigureCA -help | grep _url
-ca_domain_url <string> CA Subject Names Panel -
'https://<ca_ee_hostname>:<ca_ee_port>' URL to EE CA used to
Issue Certificates for Creation of this CA Instance (optional
but recommended for IP Port Separation)
# pkisilent ConfigureDRM -help | grep _url
-ca_domain_url <string> DRM Subject Names Panel -
'https://<ca_ee_hostname>:<ca_ee_port>' URL to EE CA used to
Issue Certificates for Creation of this DRM Instance (optional
but recommended for IP Port Separation)
# pkisilent ConfigureOCSP -help | grep _url
-ca_domain_url <string> OCSP Subject Names Panel -
'https://<ca_ee_hostname>:<ca_ee_port>' URL to EE CA used to
Issue Certificates for Creation of this OCSP Instance
(optional but recommended for IP Port Separation)
# pkisilent ConfigureTKS -help | grep _url
-ca_domain_url <string> TKS Subject Names Panel -
'https://<ca_ee_hostname>:<ca_ee_port>' URL to EE CA used to
Issue Certificates for Creation of this TKS Instance (optional
but recommended for IP Port Separation)
# pkisilent ConfigureRA -help | grep _url
-ca_issuance_url <string> CA Choice Panel -
'https://<ca_ee_hostname>:<ca_ee_port>' URL to EE CA used to
Issue Certificates (optional but recommended if used with IP
Port Separated CA)
-ca_domain_url <string> RA Subject Names Panel -
'https://<ca_ee_hostname>:<ca_ee_port>' URL to EE CA used to
Issue Certificates for Creation of this RA Instance (optional
but recommended if used with IP Port Separated CA)
# pkisilent ConfigureTPS -help | grep _url
-ca_issuance_url <string> CA Choice Panel -
'https://<ca_ee_hostname>:<ca_ee_port>' URL to EE CA
registered in this security domain used to Issue Certificates
for use by an ESC (optional but recommended if used with IP
Port Separated CA)
-tks_key_management_url <string> TKS Choice Panel -
'https://<tks_agent_hostname>:<tks_agent_port>' URL to Agent
TKS used for Key Management (optional but recommended if used
with IP Port Separated TKS)
-drm_server_side_keygen_url <string> DRM Choice Panel -
'https://<drm_agent_hostname>:<drm_agent_port>' URL to Agent
DRM used for Server-Side Keygen (optional but recommended if
used with IP Port Separated DRM)
-ca_domain_url <string> TPS Subject Names Panel -
'https://<ca_ee_hostname>:<ca_ee_port>' URL to EE CA used to
Issue Certificates for Creation of this TPS Instance (optional
but recommended if used with IP Port Separated CA)
# pkisilent ConfigureSubCA -help | grep _url
-ca_domain_url <string> SubCA Subject Names Panel -
'https://<ca_ee_hostname>:<ca_ee_port>' URL to EE CA used to
Issue Certificates for Creation of this SubCA Instance
(optional but recommended for IP Port Separation)
_______________________________________________
Pki-devel mailing list
Pki-devel(a)redhat.com
https://www.redhat.com/mailman/listinfo/pki-devel