summaryrefslogtreecommitdiffstats
path: root/base/java-tools
diff options
context:
space:
mode:
authorJack Magne <jmagne@localhost.localdomain>2015-05-06 16:49:59 -0700
committerJack Magne <jmagne@localhost.localdomain>2015-05-07 15:46:21 -0700
commit73fb345e06b70d23a852743e4dc81ef6063e738a (patch)
tree520bab7b169746ceecda50e47540193b9f041746 /base/java-tools
parent141fc3c9fdfd87b9726a87493331ad6585488461 (diff)
downloadpki-73fb345e06b70d23a852743e4dc81ef6063e738a.tar.gz
pki-73fb345e06b70d23a852743e4dc81ef6063e738a.tar.xz
pki-73fb345e06b70d23a852743e4dc81ef6063e738a.zip
Fix #1351 pki securitydomain-get-install-token fails when run with caadmin user.
The short term solution to this problem was to remove the man page information and all references to the command line module reponsible for this issue. The installer already has an alternative method to remove a subsystem from the security domain list. We now assume the alternate method and don't even try to find the token at this point. A user at the command line of the pki command will no longer be able to attempt this as well. Tested this to verify that the man page for the "securtydomain" command no longer mentions or documents the "get-install-token" variant. Tested to verify that this command can't be manually called from the command line using "pki". This attempt results in an "unknown module". Tested by installing and uninstalling a subsytem. The security domain was kept up to date as expected for each install over remove attempted.
Diffstat (limited to 'base/java-tools')
-rw-r--r--base/java-tools/man/man1/pki-securitydomain.110
-rw-r--r--base/java-tools/src/com/netscape/cmstools/system/SecurityDomainCLI.java1
-rw-r--r--base/java-tools/src/com/netscape/cmstools/system/SecurityDomainGetInstallTokenCLI.java98
3 files changed, 1 insertions, 108 deletions
diff --git a/base/java-tools/man/man1/pki-securitydomain.1 b/base/java-tools/man/man1/pki-securitydomain.1
index a8c36c556..cbefa7568 100644
--- a/base/java-tools/man/man1/pki-securitydomain.1
+++ b/base/java-tools/man/man1/pki-securitydomain.1
@@ -20,7 +20,6 @@ pki-securitydomain \- Command-Line Interface for managing Certificate System sec
.SH SYNOPSIS
.nf
\fBpki\fR [CLI options] \fBsecuritydomain\fR
-\fBpki\fR [CLI options] \fBsecuritydomain-get-install-token\fR [command options]
\fBpki\fR [CLI options] \fBsecuritydomain-show\fR [command options]
.fi
@@ -33,11 +32,6 @@ The \fBpki-securitydomain\fR commands provide command-line interfaces to manage
This command is to list available security domain commands.
.RE
.PP
-\fBpki\fR [CLI options] \fBsecuritydomain-get-install-token\fR [command options]
-.RS 4
-This command is to get an installation token.
-.RE
-.PP
\fBpki\fR [CLI options] \fBsecuritydomain-show\fR [command options]
.RS 4
This command is to show the contents of the security domain.
@@ -49,9 +43,7 @@ The CLI options are described in \fBpki\fR(1).
.SH OPERATIONS
To view available security domain commands, type \fBpki securitydomain\fP. To view each command's usage, type \fB pki securitydomain-<command> \-\-help\fP.
-To get an installation token (used when installing a new subsystem within a security domain):
-
-\fBpki <security domain admin authentication> securitydomain-get-install-token \-\-hostname <hostname> \-\-subsystem <subsystem>\fP
+." To get an installation token (used when installing a new subsystem within a security domain):
To show the contents of the security domain:
diff --git a/base/java-tools/src/com/netscape/cmstools/system/SecurityDomainCLI.java b/base/java-tools/src/com/netscape/cmstools/system/SecurityDomainCLI.java
index 224e215a8..b1a359788 100644
--- a/base/java-tools/src/com/netscape/cmstools/system/SecurityDomainCLI.java
+++ b/base/java-tools/src/com/netscape/cmstools/system/SecurityDomainCLI.java
@@ -35,7 +35,6 @@ public class SecurityDomainCLI extends CLI {
public SecurityDomainCLI(CLI parent) {
super("securitydomain", "Security domain commands", parent);
- addModule(new SecurityDomainGetInstallTokenCLI(this));
addModule(new SecurityDomainShowCLI(this));
}
diff --git a/base/java-tools/src/com/netscape/cmstools/system/SecurityDomainGetInstallTokenCLI.java b/base/java-tools/src/com/netscape/cmstools/system/SecurityDomainGetInstallTokenCLI.java
deleted file mode 100644
index 86e722aaa..000000000
--- a/base/java-tools/src/com/netscape/cmstools/system/SecurityDomainGetInstallTokenCLI.java
+++ /dev/null
@@ -1,98 +0,0 @@
-// --- BEGIN COPYRIGHT BLOCK ---
-// This program is free software; you can redistribute it and/or modify
-// it under the terms of the GNU General Public License as published by
-// the Free Software Foundation; version 2 of the License.
-//
-// This program is distributed in the hope that it will be useful,
-// but WITHOUT ANY WARRANTY; without even the implied warranty of
-// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
-// GNU General Public License for more details.
-//
-// You should have received a copy of the GNU General Public License along
-// with this program; if not, write to the Free Software Foundation, Inc.,
-// 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
-//
-// (C) 2012 Red Hat, Inc.
-// All rights reserved.
-// --- END COPYRIGHT BLOCK ---
-
-package com.netscape.cmstools.system;
-
-import java.net.InetAddress;
-import java.util.Arrays;
-
-import org.apache.commons.cli.CommandLine;
-import org.apache.commons.cli.Option;
-
-import com.netscape.certsrv.system.InstallToken;
-import com.netscape.cmstools.cli.CLI;
-import com.netscape.cmstools.cli.MainCLI;
-
-/**
- * @author Endi S. Dewata
- */
-public class SecurityDomainGetInstallTokenCLI extends CLI {
-
- public SecurityDomainCLI securityDomainCLI;
-
- public SecurityDomainGetInstallTokenCLI(SecurityDomainCLI securityDomainCLI) {
- super("get-install-token", "Get install token", securityDomainCLI);
- this.securityDomainCLI = securityDomainCLI;
-
- createOptions();
- }
-
- public void printHelp() {
- formatter.printHelp(getFullName() + " --subsystem <subsystem> [OPTIONS...]", options);
- }
-
- public void createOptions() {
- Option option = new Option(null, "hostname", true, "Hostname");
- option.setArgName("hostname");
- options.addOption(option);
-
- option = new Option(null, "subsystem", true, "Subsystem");
- option.setArgName("subsystem");
- option.setRequired(true);
- options.addOption(option);
- }
-
- public void execute(String[] args) throws Exception {
- // Always check for "--help" prior to parsing
- if (Arrays.asList(args).contains("--help")) {
- // Display usage
- printHelp();
- System.exit(0);
- }
-
- CommandLine cmd = null;
-
- try {
- cmd = parser.parse(options, args);
-
- } catch (Exception e) {
- System.err.println("Error: " + e.getMessage());
- printHelp();
- System.exit(-1);
- }
-
- String[] cmdArgs = cmd.getArgs();
-
- if (cmdArgs.length != 0) {
- System.err.println("Error: Too many arguments specified.");
- printHelp();
- System.exit(-1);
- }
-
- String hostname = cmd.getOptionValue("hostname");
- if (hostname == null) {
- hostname = InetAddress.getLocalHost().getHostName();
- }
-
- String subsystem = cmd.getOptionValue("subsystem");
-
- InstallToken token = securityDomainCLI.securityDomainClient.getInstallToken(hostname, subsystem);
-
- MainCLI.printMessage("Install token: \"" + token.getToken() + "\"");
- }
-}