You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@solr.apache.org by "sonatype-lift[bot] (via GitHub)" <gi...@apache.org> on 2023/03/20 18:44:43 UTC
[GitHub] [solr] sonatype-lift[bot] commented on a diff in pull request #1476: SOLR-16711: Extract SolrCLI tool implementations into their own package and classes
sonatype-lift[bot] commented on code in PR #1476:
URL: https://github.com/apache/solr/pull/1476#discussion_r1142548264
##########
solr/core/src/java/org/apache/solr/util/SolrCLI.java:
##########
@@ -267,7 +155,7 @@ protected abstract void runCloudTool(CloudLegacySolrClient cloudSolrClient, Comm
OPTION_VERBOSE
};
- private static void exit(int exitStatus) {
+ public static void exit(int exitStatus) {
try {
System.exit(exitStatus);
} catch (java.lang.SecurityException secExc) {
Review Comment:
<picture><img alt="7% of developers fix this issue" src="https://lift.sonatype.com/api/commentimage/fixrate/7/display.svg"></picture>
<b>*[UnnecessarilyFullyQualified](https://errorprone.info/bugpattern/UnnecessarilyFullyQualified):</b>* This fully qualified name is unambiguous to the compiler if imported.
---
```suggestion
} catch (SecurityException secExc) {
```
---
<details><summary>âšī¸ Expand to see all <b>@sonatype-lift</b> commands</summary>
You can reply with the following commands. For example, reply with ***@sonatype-lift ignoreall*** to leave out all findings.
| **Command** | **Usage** |
| ------------- | ------------- |
| `@sonatype-lift ignore` | Leave out the above finding from this PR |
| `@sonatype-lift ignoreall` | Leave out all the existing findings from this PR |
| `@sonatype-lift exclude <file\|issue\|path\|tool>` | Exclude specified `file\|issue\|path\|tool` from Lift findings by updating your config.toml file |
**Note:** When talking to LiftBot, you need to **refresh** the page to see its response.
<sub>[Click here](https://github.com/apps/sonatype-lift/installations/new) to add LiftBot to another repo.</sub></details>
---
<b>Help us improve LIFT! (<i>Sonatype LiftBot external survey</i>)</b>
Was this a good recommendation for you? <sub><small>Answering this survey will not impact your Lift settings.</small></sub>
[â[đ Not relevant](https://www.sonatype.com/lift-comment-rating?comment=444297000&lift_comment_rating=1)â] - [â[đ Won't fix](https://www.sonatype.com/lift-comment-rating?comment=444297000&lift_comment_rating=2)â] - [â[đ Not critical, will fix](https://www.sonatype.com/lift-comment-rating?comment=444297000&lift_comment_rating=3)â] - [â[đ Critical, will fix](https://www.sonatype.com/lift-comment-rating?comment=444297000&lift_comment_rating=4)â] - [â[đ Critical, fixing now](https://www.sonatype.com/lift-comment-rating?comment=444297000&lift_comment_rating=5)â]
##########
solr/core/src/java/org/apache/solr/util/SolrCLI.java:
##########
@@ -1227,156 +918,7 @@ public String toString() {
}
}
- /** Requests health information about a specific collection in SolrCloud. */
- public static class HealthcheckTool extends SolrCloudTool {
-
- public HealthcheckTool() {
- this(CLIO.getOutStream());
- }
-
- public HealthcheckTool(PrintStream stdout) {
- super(stdout);
- }
-
- @Override
- public String getName() {
- return "healthcheck";
- }
-
- @Override
- protected void runCloudTool(CloudLegacySolrClient cloudSolrClient, CommandLine cli)
- throws Exception {
- raiseLogLevelUnlessVerbose(cli);
- String collection = cli.getOptionValue("collection");
- if (collection == null)
- throw new IllegalArgumentException(
- "Must provide a collection to run a healthcheck against!");
-
- log.debug("Running healthcheck for {}", collection);
-
- ZkStateReader zkStateReader = ZkStateReader.from(cloudSolrClient);
-
- ClusterState clusterState = zkStateReader.getClusterState();
- Set<String> liveNodes = clusterState.getLiveNodes();
- final DocCollection docCollection = clusterState.getCollectionOrNull(collection);
- if (docCollection == null || docCollection.getSlices() == null)
- throw new IllegalArgumentException("Collection " + collection + " not found!");
-
- Collection<Slice> slices = docCollection.getSlices();
- // Test http code using a HEAD request first, fail fast if authentication failure
- String urlForColl =
- zkStateReader.getLeaderUrl(collection, slices.stream().findFirst().get().getName(), 1000);
- attemptHttpHead(urlForColl, cloudSolrClient.getHttpClient());
-
- SolrQuery q = new SolrQuery("*:*");
- q.setRows(0);
- QueryResponse qr = cloudSolrClient.query(collection, q);
- String collErr = null;
- long docCount = -1;
- try {
- docCount = qr.getResults().getNumFound();
- } catch (Exception exc) {
- collErr = String.valueOf(exc);
- }
-
- List<Object> shardList = new ArrayList<>();
- boolean collectionIsHealthy = (docCount != -1);
-
- for (Slice slice : slices) {
- String shardName = slice.getName();
- // since we're reporting health of this shard, there's no guarantee of a leader
- String leaderUrl = null;
- try {
- leaderUrl = zkStateReader.getLeaderUrl(collection, shardName, 1000);
- } catch (Exception exc) {
- log.warn("Failed to get leader for shard {} due to: {}", shardName, exc);
- }
-
- List<ReplicaHealth> replicaList = new ArrayList<>();
- for (Replica r : slice.getReplicas()) {
-
- String uptime = null;
- String memory = null;
- String replicaStatus = null;
- long numDocs = -1L;
-
- ZkCoreNodeProps replicaCoreProps = new ZkCoreNodeProps(r);
- String coreUrl = replicaCoreProps.getCoreUrl();
- boolean isLeader = coreUrl.equals(leaderUrl);
-
- // if replica's node is not live, its status is DOWN
- String nodeName = replicaCoreProps.getNodeName();
- if (nodeName == null || !liveNodes.contains(nodeName)) {
- replicaStatus = Replica.State.DOWN.toString();
- } else {
- // query this replica directly to get doc count and assess health
- q = new SolrQuery("*:*");
- q.setRows(0);
- q.set(DISTRIB, "false");
- try (HttpSolrClient solr = new HttpSolrClient.Builder(coreUrl).build()) {
-
- String solrUrl = solr.getBaseURL();
-
- qr = solr.query(q);
- numDocs = qr.getResults().getNumFound();
-
- int lastSlash = solrUrl.lastIndexOf('/');
- String systemInfoUrl = solrUrl.substring(0, lastSlash) + "/admin/info/system";
- Map<String, Object> info = getJson(solr.getHttpClient(), systemInfoUrl, 2, true);
- uptime = uptime(asLong("/jvm/jmx/upTimeMS", info));
- String usedMemory = asString("/jvm/memory/used", info);
- String totalMemory = asString("/jvm/memory/total", info);
- memory = usedMemory + " of " + totalMemory;
-
- // if we get here, we can trust the state
- replicaStatus = replicaCoreProps.getState();
- } catch (Exception exc) {
- log.error("ERROR: {} when trying to reach: {}", exc, coreUrl);
-
- if (checkCommunicationError(exc)) {
- replicaStatus = Replica.State.DOWN.toString();
- } else {
- replicaStatus = "error: " + exc;
- }
- }
- }
-
- replicaList.add(
- new ReplicaHealth(
- shardName,
- r.getName(),
- coreUrl,
- replicaStatus,
- numDocs,
- isLeader,
- uptime,
- memory));
- }
-
- ShardHealth shardHealth = new ShardHealth(shardName, replicaList);
- if (ShardState.healthy != shardHealth.getShardState())
- collectionIsHealthy = false; // at least one shard is un-healthy
-
- shardList.add(shardHealth.asMap());
- }
-
- Map<String, Object> report = new LinkedHashMap<>();
- report.put("collection", collection);
- report.put("status", collectionIsHealthy ? "healthy" : "degraded");
- if (collErr != null) {
- report.put("error", collErr);
- }
- report.put("numDocs", docCount);
- report.put("numShards", slices.size());
- report.put("shards", shardList);
-
- CharArr arr = new CharArr();
- new JSONWriter(arr, 2).write(report);
- echo(arr.toString());
- }
- } // end HealthcheckTool
-
- private static final Option[] CREATE_COLLECTION_OPTIONS =
+ public static final Option[] CREATE_COLLECTION_OPTIONS =
Review Comment:
<b>*[MutablePublicArray](https://errorprone.info/bugpattern/MutablePublicArray):</b>* Non-empty arrays are mutable, so this `public static final` array is not a constant and can be modified by clients of this class. Prefer an ImmutableList, or provide an accessor method that returns a defensive copy.
---
<details><summary>âšī¸ Expand to see all <b>@sonatype-lift</b> commands</summary>
You can reply with the following commands. For example, reply with ***@sonatype-lift ignoreall*** to leave out all findings.
| **Command** | **Usage** |
| ------------- | ------------- |
| `@sonatype-lift ignore` | Leave out the above finding from this PR |
| `@sonatype-lift ignoreall` | Leave out all the existing findings from this PR |
| `@sonatype-lift exclude <file\|issue\|path\|tool>` | Exclude specified `file\|issue\|path\|tool` from Lift findings by updating your config.toml file |
**Note:** When talking to LiftBot, you need to **refresh** the page to see its response.
<sub>[Click here](https://github.com/apps/sonatype-lift/installations/new) to add LiftBot to another repo.</sub></details>
---
<b>Help us improve LIFT! (<i>Sonatype LiftBot external survey</i>)</b>
Was this a good recommendation for you? <sub><small>Answering this survey will not impact your Lift settings.</small></sub>
[â[đ Not relevant](https://www.sonatype.com/lift-comment-rating?comment=444297118&lift_comment_rating=1)â] - [â[đ Won't fix](https://www.sonatype.com/lift-comment-rating?comment=444297118&lift_comment_rating=2)â] - [â[đ Not critical, will fix](https://www.sonatype.com/lift-comment-rating?comment=444297118&lift_comment_rating=3)â] - [â[đ Critical, will fix](https://www.sonatype.com/lift-comment-rating?comment=444297118&lift_comment_rating=4)â] - [â[đ Critical, fixing now](https://www.sonatype.com/lift-comment-rating?comment=444297118&lift_comment_rating=5)â]
##########
solr/core/src/java/org/apache/solr/util/cli/RunExampleTool.java:
##########
@@ -0,0 +1,1051 @@
+package org.apache.solr.util.cli;
+
+import org.apache.commons.cli.CommandLine;
+import org.apache.commons.cli.Option;
+import org.apache.commons.exec.DefaultExecuteResultHandler;
+import org.apache.commons.exec.DefaultExecutor;
+import org.apache.commons.exec.ExecuteException;
+import org.apache.commons.exec.Executor;
+import org.apache.commons.exec.OS;
+import org.apache.commons.exec.environment.EnvironmentUtils;
+import org.apache.commons.io.FileUtils;
+import org.apache.solr.client.solrj.SolrClient;
+import org.apache.solr.client.solrj.impl.CloudSolrClient;
+import org.apache.solr.client.solrj.impl.HttpSolrClient;
+import org.apache.solr.common.SolrException;
+import org.apache.solr.util.CLIO;
+import org.apache.solr.util.SimplePostTool;
+import org.apache.solr.util.SolrCLI;
+import org.noggit.CharArr;
+import org.noggit.JSONWriter;
+
+import java.io.File;
+import java.io.IOException;
+import java.io.InputStream;
+import java.io.PrintStream;
+import java.net.Socket;
+import java.net.URL;
+import java.nio.charset.StandardCharsets;
+import java.util.Arrays;
+import java.util.Collections;
+import java.util.HashMap;
+import java.util.Locale;
+import java.util.Map;
+import java.util.Optional;
+import java.util.Scanner;
+import java.util.Set;
+import java.util.concurrent.TimeUnit;
+
+/**
+ * Supports an interactive session with the user to launch (or relaunch the -e cloud example)
+ */
+public class RunExampleTool extends ToolBase {
+
+ private static final String PROMPT_FOR_NUMBER = "Please enter %s [%d]: ";
+ private static final String PROMPT_FOR_NUMBER_IN_RANGE =
+ "Please enter %s between %d and %d [%d]: ";
+ private static final String PROMPT_NUMBER_TOO_SMALL =
+ "%d is too small! " + PROMPT_FOR_NUMBER_IN_RANGE;
+ private static final String PROMPT_NUMBER_TOO_LARGE =
+ "%d is too large! " + PROMPT_FOR_NUMBER_IN_RANGE;
+
+ protected InputStream userInput;
+ protected Executor executor;
+ protected String script;
+ protected File serverDir;
+ protected File exampleDir;
+ protected String urlScheme;
+
+ /**
+ * Default constructor used by the framework when running as a command-line application.
+ */
+ public RunExampleTool() {
+ this(null, System.in, CLIO.getOutStream());
+ }
+
+ public RunExampleTool(Executor executor, InputStream userInput, PrintStream stdout) {
+ super(stdout);
+ this.executor = (executor != null) ? executor : new DefaultExecutor();
+ this.userInput = userInput;
+ }
+
+ @Override
+ public String getName() {
+ return "run_example";
+ }
+
+ @Override
+ public Option[] getOptions() {
+ return new Option[]{
+ Option.builder("noprompt")
+ .required(false)
+ .desc(
+ "Don't prompt for input; accept all defaults when running examples that accept user input.")
+ .build(),
+ Option.builder("e")
+ .argName("NAME")
+ .hasArg()
+ .required(true)
+ .desc("Name of the example to launch, one of: cloud, techproducts, schemaless, films.")
+ .longOpt("example")
+ .build(),
+ Option.builder("script")
+ .argName("PATH")
+ .hasArg()
+ .required(false)
+ .desc("Path to the bin/solr script.")
+ .build(),
+ Option.builder("d")
+ .argName("DIR")
+ .hasArg()
+ .required(true)
+ .desc("Path to the Solr server directory.")
+ .longOpt("serverDir")
+ .build(),
+ Option.builder("force")
+ .argName("FORCE")
+ .desc("Force option in case Solr is run as root.")
+ .build(),
+ Option.builder("exampleDir")
+ .argName("DIR")
+ .hasArg()
+ .required(false)
+ .desc(
+ "Path to the Solr example directory; if not provided, ${serverDir}/../example is expected to exist.")
+ .build(),
+ Option.builder("urlScheme")
+ .argName("SCHEME")
+ .hasArg()
+ .required(false)
+ .desc("Solr URL scheme: http or https, defaults to http if not specified.")
+ .build(),
+ Option.builder("p")
+ .argName("PORT")
+ .hasArg()
+ .required(false)
+ .desc("Specify the port to start the Solr HTTP listener on; default is 8983.")
+ .longOpt("port")
+ .build(),
+ Option.builder("h")
+ .argName("HOSTNAME")
+ .hasArg()
+ .required(false)
+ .desc("Specify the hostname for this Solr instance.")
+ .longOpt("host")
+ .build(),
+ Option.builder("z")
+ .argName("ZKHOST")
+ .hasArg()
+ .required(false)
+ .desc("ZooKeeper connection string; only used when running in SolrCloud mode using -c.")
+ .longOpt("zkhost")
+ .build(),
+ Option.builder("c")
+ .required(false)
+ .desc(
+ "Start Solr in SolrCloud mode; if -z not supplied, an embedded ZooKeeper instance is started on Solr port+1000, such as 9983 if Solr is bound to 8983.")
+ .longOpt("cloud")
+ .build(),
+ Option.builder("m")
+ .argName("MEM")
+ .hasArg()
+ .required(false)
+ .desc(
+ "Sets the min (-Xms) and max (-Xmx) heap size for the JVM, such as: -m 4g results in: -Xms4g -Xmx4g; by default, this script sets the heap size to 512m.")
+ .longOpt("memory")
+ .build(),
+ Option.builder("a")
+ .argName("OPTS")
+ .hasArg()
+ .required(false)
+ .desc(
+ "Additional options to be passed to the JVM when starting example Solr server(s).")
+ .longOpt("addlopts")
+ .build()
+ };
+ }
+
+ @Override
+ public void runImpl(CommandLine cli) throws Exception {
+ this.urlScheme = cli.getOptionValue("urlScheme", "http");
+
+ serverDir = new File(cli.getOptionValue("serverDir"));
+ if (!serverDir.isDirectory())
+ throw new IllegalArgumentException(
+ "Value of -serverDir option is invalid! "
+ + serverDir.getAbsolutePath()
+ + " is not a directory!");
+
+ script = cli.getOptionValue("script");
+ if (script != null) {
+ if (!(new File(script)).isFile())
+ throw new IllegalArgumentException(
+ "Value of -script option is invalid! " + script + " not found");
+ } else {
+ File scriptFile = new File(serverDir.getParentFile(), "bin/solr");
+ if (scriptFile.isFile()) {
+ script = scriptFile.getAbsolutePath();
+ } else {
+ scriptFile = new File(serverDir.getParentFile(), "bin/solr.cmd");
+ if (scriptFile.isFile()) {
+ script = scriptFile.getAbsolutePath();
+ } else {
+ throw new IllegalArgumentException(
+ "Cannot locate the bin/solr script! Please pass -script to this application.");
+ }
+ }
+ }
+
+ exampleDir =
+ (cli.hasOption("exampleDir"))
+ ? new File(cli.getOptionValue("exampleDir"))
+ : new File(serverDir.getParent(), "example");
+ if (!exampleDir.isDirectory())
+ throw new IllegalArgumentException(
+ "Value of -exampleDir option is invalid! "
+ + exampleDir.getAbsolutePath()
+ + " is not a directory!");
+
+ echoIfVerbose(
+ "Running with\nserverDir="
+ + serverDir.getAbsolutePath()
+ + ",\nexampleDir="
+ + exampleDir.getAbsolutePath()
+ + "\nscript="
+ + script,
+ cli);
+
+ String exampleType = cli.getOptionValue("example");
+ if ("cloud".equals(exampleType)) {
+ runCloudExample(cli);
+ } else if ("techproducts".equals(exampleType)
+ || "schemaless".equals(exampleType)
+ || "films".equals(exampleType)) {
+ runExample(cli, exampleType);
+ } else {
+ throw new IllegalArgumentException(
+ "Unsupported example "
+ + exampleType
+ + "! Please choose one of: cloud, schemaless, techproducts, or films");
+ }
+ }
+
+ protected void runExample(CommandLine cli, String exampleName) throws Exception {
+ File exDir = setupExampleDir(serverDir, exampleDir, exampleName);
+ String collectionName = "schemaless".equals(exampleName) ? "gettingstarted" : exampleName;
+ String configSet =
+ "techproducts".equals(exampleName) ? "sample_techproducts_configs" : "_default";
+
+ boolean isCloudMode = cli.hasOption('c');
+ String zkHost = cli.getOptionValue('z');
+ int port = Integer.parseInt(cli.getOptionValue('p', "8983"));
+ Map<String, Object> nodeStatus =
+ startSolr(new File(exDir, "solr"), isCloudMode, cli, port, zkHost, 30);
+
+ // invoke the CreateTool
+ File configsetsDir = new File(serverDir, "solr/configsets");
+
+ String solrUrl = (String) nodeStatus.get("baseUrl");
+
+ // safe check if core / collection already exists
+ boolean alreadyExists = false;
+ if (nodeStatus.get("cloud") != null) {
+ String collectionListUrl = solrUrl + "/admin/collections?action=list";
+ if (SolrCLI.safeCheckCollectionExists(collectionListUrl, collectionName)) {
+ alreadyExists = true;
+ echo(
+ "\nWARNING: Collection '"
+ + collectionName
+ + "' already exists!\nChecked collection existence using Collections API command:\n"
+ + collectionListUrl
+ + "\n");
+ }
+ } else {
+ String coreName = collectionName;
+ String coreStatusUrl = solrUrl + "/admin/cores?action=STATUS&core=" + coreName;
+ if (SolrCLI.safeCheckCoreExists(coreStatusUrl, coreName)) {
+ alreadyExists = true;
+ echo(
+ "\nWARNING: Core '"
+ + coreName
+ + "' already exists!\nChecked core existence using Core API command:\n"
+ + coreStatusUrl
+ + "\n");
+ }
+ }
+
+ if (!alreadyExists) {
+ String[] createArgs =
+ new String[]{
+ "-name", collectionName,
+ "-shards", "1",
+ "-replicationFactor", "1",
+ "-confname", collectionName,
+ "-confdir", configSet,
+ "-configsetsDir", configsetsDir.getAbsolutePath(),
+ "-solrUrl", solrUrl
+ };
+ CreateTool createTool = new CreateTool(stdout);
+ int createCode =
+ createTool.runTool(
+ SolrCLI.processCommandLineArgs(
+ createTool.getName(),
+ SolrCLI.joinCommonAndToolOptions(createTool.getOptions()),
+ createArgs));
+ if (createCode != 0)
+ throw new Exception(
+ "Failed to create "
+ + collectionName
+ + " using command: "
+ + Arrays.asList(createArgs));
+ }
+
+ if ("techproducts".equals(exampleName) && !alreadyExists) {
+
+ File exampledocsDir = new File(exampleDir, "exampledocs");
+ if (!exampledocsDir.isDirectory()) {
+ File readOnlyExampleDir = new File(serverDir.getParentFile(), "example");
+ if (readOnlyExampleDir.isDirectory()) {
+ exampledocsDir = new File(readOnlyExampleDir, "exampledocs");
+ }
+ }
+
+ if (exampledocsDir.isDirectory()) {
+ String updateUrl = String.format(Locale.ROOT, "%s/%s/update", solrUrl, collectionName);
+ echo("Indexing tech product example docs from " + exampledocsDir.getAbsolutePath());
+
+ String currentPropVal = System.getProperty("url");
+ System.setProperty("url", updateUrl);
+ SimplePostTool.main(new String[]{exampledocsDir.getAbsolutePath() + "/*.xml"});
+ if (currentPropVal != null) {
+ System.setProperty("url", currentPropVal); // reset
+ } else {
+ System.clearProperty("url");
+ }
+ } else {
+ echo(
+ "exampledocs directory not found, skipping indexing step for the techproducts example");
+ }
+ } else if ("films".equals(exampleName) && !alreadyExists) {
+ SolrClient solrClient = new HttpSolrClient.Builder(solrUrl).build();
+
+ echo("Adding dense vector field type to films schema \"_default\"");
+ try {
+ SolrCLI.postJsonToSolr(
+ solrClient,
+ "/" + collectionName + "/schema",
+ "{\n"
+ + " \"add-field-type\" : {\n"
+ + " \"name\":\"knn_vector_10\",\n"
+ + " \"class\":\"solr.DenseVectorField\",\n"
+ + " \"vectorDimension\":10,\n"
+ + " \"similarityFunction\":cosine\n"
+ + " \"knnAlgorithm\":hnsw\n"
+ + " }\n"
+ + " }");
+ } catch (Exception ex) {
+ throw new SolrException(SolrException.ErrorCode.SERVER_ERROR, ex);
+ }
+
+ echo(
+ "Adding name, initial_release_date, and film_vector fields to films schema \"_default\"");
+ try {
+ SolrCLI.postJsonToSolr(
+ solrClient,
+ "/" + collectionName + "/schema",
+ "{\n"
+ + " \"add-field\" : {\n"
+ + " \"name\":\"name\",\n"
+ + " \"type\":\"text_general\",\n"
+ + " \"multiValued\":false,\n"
+ + " \"stored\":true\n"
+ + " },\n"
+ + " \"add-field\" : {\n"
+ + " \"name\":\"initial_release_date\",\n"
+ + " \"type\":\"pdate\",\n"
+ + " \"stored\":true\n"
+ + " },\n"
+ + " \"add-field\" : {\n"
+ + " \"name\":\"film_vector\",\n"
+ + " \"type\":\"knn_vector_10\",\n"
+ + " \"indexed\":true\n"
+ + " \"stored\":true\n"
+ + " }\n"
+ + " }");
+ } catch (Exception ex) {
+ throw new SolrException(SolrException.ErrorCode.SERVER_ERROR, ex);
+ }
+
+ echo(
+ "Adding paramsets \"algo\" and \"algo_b\" to films configuration for relevancy tuning");
+ try {
+ SolrCLI.postJsonToSolr(
+ solrClient,
+ "/" + collectionName + "/config/params",
+ "{\n"
+ + " \"set\": {\n"
+ + " \"algo_a\":{\n"
+ + " \"defType\":\"dismax\",\n"
+ + " \"qf\":\"name\"\n"
+ + " }\n"
+ + " },\n"
+ + " \"set\": {\n"
+ + " \"algo_b\":{\n"
+ + " \"defType\":\"dismax\",\n"
+ + " \"qf\":\"name\",\n"
+ + " \"mm\":\"100%\"\n"
+ + " }\n"
+ + " }\n"
+ + " }\n");
+ } catch (Exception ex) {
+ throw new SolrException(SolrException.ErrorCode.SERVER_ERROR, ex);
+ }
+
+ File filmsJsonFile = new File(exampleDir, "films/films.json");
+ String updateUrl = String.format(Locale.ROOT, "%s/%s/update/json", solrUrl, collectionName);
+ echo("Indexing films example docs from " + filmsJsonFile.getAbsolutePath());
+ String currentPropVal = System.getProperty("url");
+ System.setProperty("url", updateUrl);
+ SimplePostTool.main(new String[]{filmsJsonFile.getAbsolutePath()});
+ if (currentPropVal != null) {
+ System.setProperty("url", currentPropVal); // reset
+ } else {
+ System.clearProperty("url");
+ }
+ }
+
+ echo(
+ "\nSolr "
+ + exampleName
+ + " example launched successfully. Direct your Web browser to "
+ + solrUrl
+ + " to visit the Solr Admin UI");
+ }
+
+ protected void runCloudExample(CommandLine cli) throws Exception {
+
+ boolean prompt = !cli.hasOption("noprompt");
+ int numNodes = 2;
+ int[] cloudPorts = new int[]{8983, 7574, 8984, 7575};
+ File cloudDir = new File(exampleDir, "cloud");
+ if (!cloudDir.isDirectory()) {
+ cloudDir.mkdir();
+ }
+
+ echo("\nWelcome to the SolrCloud example!\n");
+
+ Scanner readInput = prompt ? new Scanner(userInput, StandardCharsets.UTF_8.name()) : null;
+ if (prompt) {
+ echo(
+ "This interactive session will help you launch a SolrCloud cluster on your local workstation.");
+
+ // get the number of nodes to start
+ numNodes =
+ promptForInt(
+ readInput,
+ "To begin, how many Solr nodes would you like to run in your local cluster? (specify 1-4 nodes) [2]: ",
+ "a number",
+ numNodes,
+ 1,
+ 4);
+
+ echo("Ok, let's start up " + numNodes + " Solr nodes for your example SolrCloud cluster.");
+
+ // get the ports for each port
+ for (int n = 0; n < numNodes; n++) {
+ String promptMsg =
+ String.format(
+ Locale.ROOT, "Please enter the port for node%d [%d]: ", (n + 1), cloudPorts[n]);
+ int port = promptForPort(readInput, n + 1, promptMsg, cloudPorts[n]);
+ while (!isPortAvailable(port)) {
+ port =
+ promptForPort(
+ readInput,
+ n + 1,
+ "Oops! Looks like port "
+ + port
+ + " is already being used by another process. Please choose a different port.",
+ cloudPorts[n]);
+ }
+
+ cloudPorts[n] = port;
+ echoIfVerbose("Using port " + port + " for node " + (n + 1), cli);
+ }
+ } else {
+ echo("Starting up " + numNodes + " Solr nodes for your example SolrCloud cluster.\n");
+ }
+
+ // setup a unique solr.solr.home directory for each node
+ File node1Dir = setupExampleDir(serverDir, cloudDir, "node1");
+ for (int n = 2; n <= numNodes; n++) {
+ File nodeNDir = new File(cloudDir, "node" + n);
+ if (!nodeNDir.isDirectory()) {
+ echo("Cloning " + node1Dir.getAbsolutePath() + " into\n " + nodeNDir.getAbsolutePath());
+ FileUtils.copyDirectory(node1Dir, nodeNDir);
+ } else {
+ echo(nodeNDir.getAbsolutePath() + " already exists.");
+ }
+ }
+
+ // deal with extra args passed to the script to run the example
+ String zkHost = cli.getOptionValue('z');
+
+ // start the first node (most likely with embedded ZK)
+ Map<String, Object> nodeStatus =
+ startSolr(new File(node1Dir, "solr"), true, cli, cloudPorts[0], zkHost, 30);
+
+ if (zkHost == null) {
+ @SuppressWarnings("unchecked")
+ Map<String, Object> cloudStatus = (Map<String, Object>) nodeStatus.get("cloud");
+ if (cloudStatus != null) {
+ String zookeeper = (String) cloudStatus.get("ZooKeeper");
+ if (zookeeper != null) zkHost = zookeeper;
+ }
+ if (zkHost == null)
+ throw new Exception("Could not get the ZooKeeper connection string for node1!");
+ }
+
+ if (numNodes > 1) {
+ // start the other nodes
+ for (int n = 1; n < numNodes; n++)
+ startSolr(
+ new File(cloudDir, "node" + (n + 1) + "/solr"), true, cli, cloudPorts[n], zkHost, 30);
+ }
+
+ String solrUrl = (String) nodeStatus.get("baseUrl");
+ if (solrUrl.endsWith("/")) solrUrl = solrUrl.substring(0, solrUrl.length() - 1);
+
+ // wait until live nodes == numNodes
+ waitToSeeLiveNodes(10 /* max wait */, zkHost, numNodes);
+
+ // create the collection
+ String collectionName = createCloudExampleCollection(numNodes, readInput, prompt, solrUrl);
+
+ // update the config to enable soft auto-commit
+ echo("\nEnabling auto soft-commits with maxTime 3 secs using the Config API");
+ setCollectionConfigProperty(
+ solrUrl, collectionName, "updateHandler.autoSoftCommit.maxTime", "3000");
+
+ echo("\n\nSolrCloud example running, please visit: " + solrUrl + " \n");
+ }
+
+ protected void setCollectionConfigProperty(
+ String solrUrl, String collectionName, String propName, String propValue) {
+ ConfigTool configTool = new ConfigTool(stdout);
+ String[] configArgs =
+ new String[]{
+ "-collection",
+ collectionName,
+ "-property",
+ propName,
+ "-value",
+ propValue,
+ "-solrUrl",
+ solrUrl
+ };
+
+ // let's not fail if we get this far ... just report error and finish up
+ try {
+ configTool.runTool(
+ SolrCLI.processCommandLineArgs(
+ configTool.getName(),
+ SolrCLI.joinCommonAndToolOptions(configTool.getOptions()),
+ configArgs));
+ } catch (Exception exc) {
+ CLIO.err("Failed to update '" + propName + "' property due to: " + exc);
+ }
+ }
+
+ protected void waitToSeeLiveNodes(int maxWaitSecs, String zkHost, int numNodes) {
+ try (CloudSolrClient cloudClient =
+ new CloudSolrClient.Builder(Collections.singletonList(zkHost), Optional.empty())
+ .build()){
+
+ cloudClient.connect();
+ Set<String> liveNodes = cloudClient.getClusterState().getLiveNodes();
+ int numLiveNodes = (liveNodes != null) ? liveNodes.size() : 0;
+ long timeout =
+ System.nanoTime() + TimeUnit.NANOSECONDS.convert(maxWaitSecs, TimeUnit.SECONDS);
+ while (System.nanoTime() < timeout && numLiveNodes < numNodes) {
+ echo(
+ "\nWaiting up to "
+ + maxWaitSecs
+ + " seconds to see "
+ + (numNodes - numLiveNodes)
+ + " more nodes join the SolrCloud cluster ...");
+ try {
+ Thread.sleep(2000);
+ } catch (InterruptedException ie) {
+ Thread.interrupted();
+ }
+ liveNodes = cloudClient.getClusterState().getLiveNodes();
+ numLiveNodes = (liveNodes != null) ? liveNodes.size() : 0;
+ }
+ if (numLiveNodes < numNodes) {
+ echo(
+ "\nWARNING: Only "
+ + numLiveNodes
+ + " of "
+ + numNodes
+ + " are active in the cluster after "
+ + maxWaitSecs
+ + " seconds! Please check the solr.log for each node to look for errors.\n");
+ }
+ } catch (Exception exc) {
+ CLIO.err("Failed to see if " + numNodes + " joined the SolrCloud cluster due to: " + exc);
+ }
+ }
+
+ protected Map<String, Object> startSolr(
+ File solrHomeDir,
+ boolean cloudMode,
+ CommandLine cli,
+ int port,
+ String zkHost,
+ int maxWaitSecs)
+ throws Exception {
+
+ String extraArgs = readExtraArgs(cli.getArgs());
+
+ String host = cli.getOptionValue('h');
+ String memory = cli.getOptionValue('m');
+
+ String hostArg = (host != null && !"localhost".equals(host)) ? " -h " + host : "";
+ String zkHostArg = (zkHost != null) ? " -z " + zkHost : "";
+ String memArg = (memory != null) ? " -m " + memory : "";
+ String cloudModeArg = cloudMode ? "-cloud " : "";
+ String forceArg = cli.hasOption("force") ? " -force" : "";
+
+ String addlOpts = cli.getOptionValue('a');
+ String addlOptsArg = (addlOpts != null) ? " -a \"" + addlOpts + "\"" : "";
+
+ File cwd = new File(System.getProperty("user.dir"));
+ File binDir = (new File(script)).getParentFile();
+
+ boolean isWindows = (OS.isFamilyDOS() || OS.isFamilyWin9x() || OS.isFamilyWindows());
+ String callScript = (!isWindows && cwd.equals(binDir.getParentFile())) ? "bin/solr" : script;
+
+ String cwdPath = cwd.getAbsolutePath();
+ String solrHome = solrHomeDir.getAbsolutePath();
+
+ // don't display a huge path for solr home if it is relative to the cwd
+ if (!isWindows && cwdPath.length() > 1 && solrHome.startsWith(cwdPath))
+ solrHome = solrHome.substring(cwdPath.length() + 1);
+
+ String startCmd =
+ String.format(
+ Locale.ROOT,
+ "\"%s\" start %s -p %d -s \"%s\" %s %s %s %s %s %s",
+ callScript,
+ cloudModeArg,
+ port,
+ solrHome,
+ hostArg,
+ zkHostArg,
+ memArg,
+ forceArg,
+ extraArgs,
+ addlOptsArg);
+ startCmd = startCmd.replaceAll("\\s+", " ").trim(); // for pretty printing
+
+ echo("\nStarting up Solr on port " + port + " using command:");
+ echo(startCmd + "\n");
+
+ String solrUrl =
+ String.format(
+ Locale.ROOT, "%s://%s:%d/solr", urlScheme, (host != null ? host : "localhost"), port);
+
+ Map<String, Object> nodeStatus = checkPortConflict(solrUrl, solrHomeDir, port);
+ if (nodeStatus != null)
+ return nodeStatus; // the server they are trying to start is already running
+
+ int code = 0;
+ if (isWindows) {
+ // On Windows, the execution doesn't return, so we have to execute async
+ // and when calling the script, it seems to be inheriting the environment that launched this
+ // app, so we have to prune out env vars that may cause issues
+ Map<String, String> startEnv = new HashMap<>();
+ Map<String, String> procEnv = EnvironmentUtils.getProcEnvironment();
+ if (procEnv != null) {
+ for (Map.Entry<String, String> entry : procEnv.entrySet()) {
+ String envVar = entry.getKey();
+ String envVarVal = entry.getValue();
+ if (envVarVal != null && !"EXAMPLE".equals(envVar) && !envVar.startsWith("SOLR_")) {
+ startEnv.put(envVar, envVarVal);
+ }
+ }
+ }
+ DefaultExecuteResultHandler handler = new DefaultExecuteResultHandler();
+ executor.execute(org.apache.commons.exec.CommandLine.parse(startCmd), startEnv, handler);
+
+ // wait for execution.
+ try {
+ handler.waitFor(3000);
+ } catch (InterruptedException ie) {
+ // safe to ignore ...
+ Thread.interrupted();
+ }
+ if (handler.hasResult() && handler.getExitValue() != 0) {
+ throw new Exception(
+ "Failed to start Solr using command: "
+ + startCmd
+ + " Exception : "
+ + handler.getException());
+ }
+ } else {
+ try {
+ code = executor.execute(org.apache.commons.exec.CommandLine.parse(startCmd));
+ } catch (ExecuteException e) {
+ throw new Exception(
+ "Failed to start Solr using command: " + startCmd + " Exception : " + e);
+ }
+ }
+ if (code != 0) throw new Exception("Failed to start Solr using command: " + startCmd);
+
+ return getNodeStatus(solrUrl, maxWaitSecs);
+ }
+
+ protected Map<String, Object> checkPortConflict(
+ String solrUrl, File solrHomeDir, int port) {
+ // quickly check if the port is in use
+ if (isPortAvailable(port)) return null; // not in use ... try to start
+
+ Map<String, Object> nodeStatus = null;
+ try {
+ nodeStatus = (new StatusTool()).getStatus(solrUrl);
+ } catch (Exception ignore) {
+ /* just trying to determine if this example is already running. */
+ }
+
+ if (nodeStatus != null) {
+ String solr_home = (String) nodeStatus.get("solr_home");
+ if (solr_home != null) {
+ String solrHomePath = solrHomeDir.getAbsolutePath();
+ if (!solrHomePath.endsWith("/")) solrHomePath += "/";
+ if (!solr_home.endsWith("/")) solr_home += "/";
+
+ if (solrHomePath.equals(solr_home)) {
+ CharArr arr = new CharArr();
+ new JSONWriter(arr, 2).write(nodeStatus);
+ echo(
+ "Solr is already setup and running on port "
+ + port
+ + " with status:\n"
+ + arr.toString());
+ echo(
+ "\nIf this is not the example node you are trying to start, please choose a different port.");
+ nodeStatus.put("baseUrl", solrUrl);
+ return nodeStatus;
+ }
+ }
+ }
+
+ throw new IllegalStateException(
+ "Port " + port + " is already being used by another process.");
+ }
+
+ protected String readExtraArgs(String[] extraArgsArr) {
+ String extraArgs = "";
+ if (extraArgsArr != null && extraArgsArr.length > 0) {
+ StringBuilder sb = new StringBuilder();
+ int app = 0;
+ for (int e = 0; e < extraArgsArr.length; e++) {
+ String arg = extraArgsArr[e];
+ if ("e".equals(arg) || "example".equals(arg)) {
+ e++; // skip over the example arg
+ continue;
+ }
+
+ if (app > 0) sb.append(" ");
+ sb.append(arg);
+ ++app;
+ }
+ extraArgs = sb.toString().trim();
+ }
+ return extraArgs;
+ }
+
+ protected String createCloudExampleCollection(
+ int numNodes, Scanner readInput, boolean prompt, String solrUrl) throws Exception {
+ // yay! numNodes SolrCloud nodes running
+ int numShards = 2;
+ int replicationFactor = 2;
+ String cloudConfig = "_default";
+ String collectionName = "gettingstarted";
+
+ File configsetsDir = new File(serverDir, "solr/configsets");
+ String collectionListUrl = solrUrl + "/admin/collections?action=list";
+
+ if (prompt) {
+ echo(
+ "\nNow let's create a new collection for indexing documents in your "
+ + numNodes
+ + "-node cluster.");
+
+ while (true) {
+ collectionName =
+ prompt(
+ readInput,
+ "Please provide a name for your new collection: [" + collectionName + "] ",
+ collectionName);
+
+ // Test for existence and then prompt to either create another collection or skip the creation step
+ if (SolrCLI.safeCheckCollectionExists(collectionListUrl, collectionName)) {
+ echo("\nCollection '" + collectionName + "' already exists!");
+ int oneOrTwo =
+ promptForInt(
+ readInput,
+ "Do you want to re-use the existing collection or create a new one? Enter 1 to reuse, 2 to create new [1]: ",
+ "a 1 or 2",
+ 1,
+ 1,
+ 2);
+ if (oneOrTwo == 1) {
+ return collectionName;
+ } else {
+ continue;
+ }
+ } else {
+ break; // user selected a collection that doesn't exist ... proceed on
+ }
+ }
+
+ numShards =
+ promptForInt(
+ readInput,
+ "How many shards would you like to split " + collectionName + " into? [2]",
+ "a shard count",
+ 2,
+ 1,
+ 4);
+
+ replicationFactor =
+ promptForInt(
+ readInput,
+ "How many replicas per shard would you like to create? [2] ",
+ "a replication factor",
+ 2,
+ 1,
+ 4);
+
+ echo(
+ "Please choose a configuration for the "
+ + collectionName
+ + " collection, available options are:");
+ String validConfigs = "_default or sample_techproducts_configs [" + cloudConfig + "] ";
+ cloudConfig = prompt(readInput, validConfigs, cloudConfig);
+
+ // validate the cloudConfig name
+ while (!isValidConfig(configsetsDir, cloudConfig)) {
+ echo(
+ cloudConfig
+ + " is not a valid configuration directory! Please choose a configuration for the "
+ + collectionName
+ + " collection, available options are:");
+ cloudConfig = prompt(readInput, validConfigs, cloudConfig);
+ }
+ } else {
+ // must verify if default collection exists
+ if (SolrCLI.safeCheckCollectionExists(collectionListUrl, collectionName)) {
+ echo(
+ "\nCollection '"
+ + collectionName
+ + "' already exists! Skipping collection creation step.");
+ return collectionName;
+ }
+ }
+
+ // invoke the CreateCollectionTool
+ String[] createArgs =
+ new String[]{
+ "-name", collectionName,
+ "-shards", String.valueOf(numShards),
+ "-replicationFactor", String.valueOf(replicationFactor),
+ "-confname", collectionName,
+ "-confdir", cloudConfig,
+ "-configsetsDir", configsetsDir.getAbsolutePath(),
+ "-solrUrl", solrUrl
+ };
+
+ CreateCollectionTool createCollectionTool = new CreateCollectionTool(stdout);
+ int createCode =
+ createCollectionTool.runTool(
+ SolrCLI.processCommandLineArgs(
+ createCollectionTool.getName(),
+ SolrCLI.joinCommonAndToolOptions(createCollectionTool.getOptions()),
+ createArgs));
+
+ if (createCode != 0)
+ throw new Exception(
+ "Failed to create collection using command: " + Arrays.asList(createArgs));
+
+ return collectionName;
+ }
+
+ protected boolean isValidConfig(File configsetsDir, String config) {
+ File configDir = new File(configsetsDir, config);
+ if (configDir.isDirectory()) return true;
+
+ // not a built-in configset ... maybe it's a custom directory?
+ configDir = new File(config);
+ return configDir.isDirectory();
+ }
+
+ protected Map<String, Object> getNodeStatus(String solrUrl, int maxWaitSecs) throws Exception {
+ StatusTool statusTool = new StatusTool();
+ if (verbose) echo("\nChecking status of Solr at " + solrUrl + " ...");
+
+ URL solrURL = new URL(solrUrl);
+ Map<String, Object> nodeStatus = statusTool.waitToSeeSolrUp(solrUrl, maxWaitSecs);
+ nodeStatus.put("baseUrl", solrUrl);
+ CharArr arr = new CharArr();
+ new JSONWriter(arr, 2).write(nodeStatus);
+ String mode = (nodeStatus.get("cloud") != null) ? "cloud" : "standalone";
+ if (verbose)
+ echo(
+ "\nSolr is running on "
+ + solrURL.getPort()
+ + " in "
+ + mode
+ + " mode with status:\n"
+ + arr);
+
+ return nodeStatus;
+ }
+
+ protected File setupExampleDir(File serverDir, File exampleParentDir, String dirName)
+ throws IOException {
+ File solrXml = new File(serverDir, "solr/solr.xml");
+ if (!solrXml.isFile())
+ throw new IllegalArgumentException(
+ "Value of -serverDir option is invalid! " + solrXml.getAbsolutePath() + " not found!");
+
+ File zooCfg = new File(serverDir, "solr/zoo.cfg");
+ if (!zooCfg.isFile())
+ throw new IllegalArgumentException(
+ "Value of -serverDir option is invalid! " + zooCfg.getAbsolutePath() + " not found!");
+
+ File solrHomeDir = new File(exampleParentDir, dirName + "/solr");
+ if (!solrHomeDir.isDirectory()) {
+ echo("Creating Solr home directory " + solrHomeDir);
+ solrHomeDir.mkdirs();
+ } else {
+ echo("Solr home directory " + solrHomeDir.getAbsolutePath() + " already exists.");
+ }
+
+ copyIfNeeded(solrXml, new File(solrHomeDir, "solr.xml"));
+ copyIfNeeded(zooCfg, new File(solrHomeDir, "zoo.cfg"));
+
+ return solrHomeDir.getParentFile();
+ }
+
+ protected void copyIfNeeded(File src, File dest) throws IOException {
+ if (!dest.isFile()) FileUtils.copyFile(src, dest);
+
+ if (!dest.isFile())
+ throw new IllegalStateException("Required file " + dest.getAbsolutePath() + " not found!");
+ }
+
+ protected boolean isPortAvailable(int port) {
+ Socket s = null;
+ try {
+ s = new Socket("localhost", port);
Review Comment:
<b>*[UNENCRYPTED_SOCKET](https://find-sec-bugs.github.io/bugs.htm#UNENCRYPTED_SOCKET):</b>* Unencrypted socket to org.apache.solr.util.cli.RunExampleTool (instead of SSLSocket)
---
<details><summary>âšī¸ Expand to see all <b>@sonatype-lift</b> commands</summary>
You can reply with the following commands. For example, reply with ***@sonatype-lift ignoreall*** to leave out all findings.
| **Command** | **Usage** |
| ------------- | ------------- |
| `@sonatype-lift ignore` | Leave out the above finding from this PR |
| `@sonatype-lift ignoreall` | Leave out all the existing findings from this PR |
| `@sonatype-lift exclude <file\|issue\|path\|tool>` | Exclude specified `file\|issue\|path\|tool` from Lift findings by updating your config.toml file |
**Note:** When talking to LiftBot, you need to **refresh** the page to see its response.
<sub>[Click here](https://github.com/apps/sonatype-lift/installations/new) to add LiftBot to another repo.</sub></details>
---
<b>Help us improve LIFT! (<i>Sonatype LiftBot external survey</i>)</b>
Was this a good recommendation for you? <sub><small>Answering this survey will not impact your Lift settings.</small></sub>
[â[đ Not relevant](https://www.sonatype.com/lift-comment-rating?comment=444297464&lift_comment_rating=1)â] - [â[đ Won't fix](https://www.sonatype.com/lift-comment-rating?comment=444297464&lift_comment_rating=2)â] - [â[đ Not critical, will fix](https://www.sonatype.com/lift-comment-rating?comment=444297464&lift_comment_rating=3)â] - [â[đ Critical, will fix](https://www.sonatype.com/lift-comment-rating?comment=444297464&lift_comment_rating=4)â] - [â[đ Critical, fixing now](https://www.sonatype.com/lift-comment-rating?comment=444297464&lift_comment_rating=5)â]
##########
solr/core/src/java/org/apache/solr/util/cli/ToolBase.java:
##########
@@ -0,0 +1,55 @@
+package org.apache.solr.util.cli;
+
+import org.apache.commons.cli.CommandLine;
+import org.apache.solr.util.CLIO;
+import org.apache.solr.util.SolrCLI;
+
+import java.io.PrintStream;
+
+public abstract class ToolBase implements Tool {
+ protected PrintStream stdout;
+ protected boolean verbose = false;
+
+ protected ToolBase() {
+ this(CLIO.getOutStream());
+ }
+
+ protected ToolBase(PrintStream stdout) {
+ this.stdout = stdout;
+ }
+
+ protected void echoIfVerbose(final String msg, CommandLine cli) {
+ if (cli.hasOption(SolrCLI.OPTION_VERBOSE.getOpt())) {
+ echo(msg);
+ }
+ }
+
+ protected void echo(final String msg) {
+ stdout.println(msg);
+ }
+
+ @Override
+ public int runTool(CommandLine cli) throws Exception {
+ verbose = cli.hasOption(SolrCLI.OPTION_VERBOSE.getOpt());
+
+ int toolExitStatus = 0;
+ try {
+ runImpl(cli);
+ } catch (Exception exc) {
+ // since this is a CLI, spare the user the stacktrace
+ String excMsg = exc.getMessage();
+ if (excMsg != null) {
+ CLIO.err("\nERROR: " + excMsg + "\n");
+ if (verbose) {
+ exc.printStackTrace(CLIO.getErrStream());
Review Comment:
<picture><img alt="0% of developers fix this issue" src="https://lift.sonatype.com/api/commentimage/fixrate/0/display.svg"></picture>
<b>*[INFORMATION_EXPOSURE_THROUGH_AN_ERROR_MESSAGE](https://find-sec-bugs.github.io/bugs.htm#INFORMATION_EXPOSURE_THROUGH_AN_ERROR_MESSAGE):</b>* Possible information exposure through an error message
---
<details><summary>âšī¸ Expand to see all <b>@sonatype-lift</b> commands</summary>
You can reply with the following commands. For example, reply with ***@sonatype-lift ignoreall*** to leave out all findings.
| **Command** | **Usage** |
| ------------- | ------------- |
| `@sonatype-lift ignore` | Leave out the above finding from this PR |
| `@sonatype-lift ignoreall` | Leave out all the existing findings from this PR |
| `@sonatype-lift exclude <file\|issue\|path\|tool>` | Exclude specified `file\|issue\|path\|tool` from Lift findings by updating your config.toml file |
**Note:** When talking to LiftBot, you need to **refresh** the page to see its response.
<sub>[Click here](https://github.com/apps/sonatype-lift/installations/new) to add LiftBot to another repo.</sub></details>
---
<b>Help us improve LIFT! (<i>Sonatype LiftBot external survey</i>)</b>
Was this a good recommendation for you? <sub><small>Answering this survey will not impact your Lift settings.</small></sub>
[â[đ Not relevant](https://www.sonatype.com/lift-comment-rating?comment=444297528&lift_comment_rating=1)â] - [â[đ Won't fix](https://www.sonatype.com/lift-comment-rating?comment=444297528&lift_comment_rating=2)â] - [â[đ Not critical, will fix](https://www.sonatype.com/lift-comment-rating?comment=444297528&lift_comment_rating=3)â] - [â[đ Critical, will fix](https://www.sonatype.com/lift-comment-rating?comment=444297528&lift_comment_rating=4)â] - [â[đ Critical, fixing now](https://www.sonatype.com/lift-comment-rating?comment=444297528&lift_comment_rating=5)â]
##########
solr/core/src/java/org/apache/solr/util/cli/RunExampleTool.java:
##########
@@ -0,0 +1,1051 @@
+package org.apache.solr.util.cli;
+
+import org.apache.commons.cli.CommandLine;
+import org.apache.commons.cli.Option;
+import org.apache.commons.exec.DefaultExecuteResultHandler;
+import org.apache.commons.exec.DefaultExecutor;
+import org.apache.commons.exec.ExecuteException;
+import org.apache.commons.exec.Executor;
+import org.apache.commons.exec.OS;
+import org.apache.commons.exec.environment.EnvironmentUtils;
+import org.apache.commons.io.FileUtils;
+import org.apache.solr.client.solrj.SolrClient;
+import org.apache.solr.client.solrj.impl.CloudSolrClient;
+import org.apache.solr.client.solrj.impl.HttpSolrClient;
+import org.apache.solr.common.SolrException;
+import org.apache.solr.util.CLIO;
+import org.apache.solr.util.SimplePostTool;
+import org.apache.solr.util.SolrCLI;
+import org.noggit.CharArr;
+import org.noggit.JSONWriter;
+
+import java.io.File;
+import java.io.IOException;
+import java.io.InputStream;
+import java.io.PrintStream;
+import java.net.Socket;
+import java.net.URL;
+import java.nio.charset.StandardCharsets;
+import java.util.Arrays;
+import java.util.Collections;
+import java.util.HashMap;
+import java.util.Locale;
+import java.util.Map;
+import java.util.Optional;
+import java.util.Scanner;
+import java.util.Set;
+import java.util.concurrent.TimeUnit;
+
+/**
+ * Supports an interactive session with the user to launch (or relaunch the -e cloud example)
+ */
+public class RunExampleTool extends ToolBase {
+
+ private static final String PROMPT_FOR_NUMBER = "Please enter %s [%d]: ";
+ private static final String PROMPT_FOR_NUMBER_IN_RANGE =
+ "Please enter %s between %d and %d [%d]: ";
+ private static final String PROMPT_NUMBER_TOO_SMALL =
+ "%d is too small! " + PROMPT_FOR_NUMBER_IN_RANGE;
+ private static final String PROMPT_NUMBER_TOO_LARGE =
+ "%d is too large! " + PROMPT_FOR_NUMBER_IN_RANGE;
+
+ protected InputStream userInput;
+ protected Executor executor;
+ protected String script;
+ protected File serverDir;
+ protected File exampleDir;
+ protected String urlScheme;
+
+ /**
+ * Default constructor used by the framework when running as a command-line application.
+ */
+ public RunExampleTool() {
+ this(null, System.in, CLIO.getOutStream());
+ }
+
+ public RunExampleTool(Executor executor, InputStream userInput, PrintStream stdout) {
+ super(stdout);
+ this.executor = (executor != null) ? executor : new DefaultExecutor();
+ this.userInput = userInput;
+ }
+
+ @Override
+ public String getName() {
+ return "run_example";
+ }
+
+ @Override
+ public Option[] getOptions() {
+ return new Option[]{
+ Option.builder("noprompt")
+ .required(false)
+ .desc(
+ "Don't prompt for input; accept all defaults when running examples that accept user input.")
+ .build(),
+ Option.builder("e")
+ .argName("NAME")
+ .hasArg()
+ .required(true)
+ .desc("Name of the example to launch, one of: cloud, techproducts, schemaless, films.")
+ .longOpt("example")
+ .build(),
+ Option.builder("script")
+ .argName("PATH")
+ .hasArg()
+ .required(false)
+ .desc("Path to the bin/solr script.")
+ .build(),
+ Option.builder("d")
+ .argName("DIR")
+ .hasArg()
+ .required(true)
+ .desc("Path to the Solr server directory.")
+ .longOpt("serverDir")
+ .build(),
+ Option.builder("force")
+ .argName("FORCE")
+ .desc("Force option in case Solr is run as root.")
+ .build(),
+ Option.builder("exampleDir")
+ .argName("DIR")
+ .hasArg()
+ .required(false)
+ .desc(
+ "Path to the Solr example directory; if not provided, ${serverDir}/../example is expected to exist.")
+ .build(),
+ Option.builder("urlScheme")
+ .argName("SCHEME")
+ .hasArg()
+ .required(false)
+ .desc("Solr URL scheme: http or https, defaults to http if not specified.")
+ .build(),
+ Option.builder("p")
+ .argName("PORT")
+ .hasArg()
+ .required(false)
+ .desc("Specify the port to start the Solr HTTP listener on; default is 8983.")
+ .longOpt("port")
+ .build(),
+ Option.builder("h")
+ .argName("HOSTNAME")
+ .hasArg()
+ .required(false)
+ .desc("Specify the hostname for this Solr instance.")
+ .longOpt("host")
+ .build(),
+ Option.builder("z")
+ .argName("ZKHOST")
+ .hasArg()
+ .required(false)
+ .desc("ZooKeeper connection string; only used when running in SolrCloud mode using -c.")
+ .longOpt("zkhost")
+ .build(),
+ Option.builder("c")
+ .required(false)
+ .desc(
+ "Start Solr in SolrCloud mode; if -z not supplied, an embedded ZooKeeper instance is started on Solr port+1000, such as 9983 if Solr is bound to 8983.")
+ .longOpt("cloud")
+ .build(),
+ Option.builder("m")
+ .argName("MEM")
+ .hasArg()
+ .required(false)
+ .desc(
+ "Sets the min (-Xms) and max (-Xmx) heap size for the JVM, such as: -m 4g results in: -Xms4g -Xmx4g; by default, this script sets the heap size to 512m.")
+ .longOpt("memory")
+ .build(),
+ Option.builder("a")
+ .argName("OPTS")
+ .hasArg()
+ .required(false)
+ .desc(
+ "Additional options to be passed to the JVM when starting example Solr server(s).")
+ .longOpt("addlopts")
+ .build()
+ };
+ }
+
+ @Override
+ public void runImpl(CommandLine cli) throws Exception {
+ this.urlScheme = cli.getOptionValue("urlScheme", "http");
+
+ serverDir = new File(cli.getOptionValue("serverDir"));
+ if (!serverDir.isDirectory())
+ throw new IllegalArgumentException(
+ "Value of -serverDir option is invalid! "
+ + serverDir.getAbsolutePath()
+ + " is not a directory!");
+
+ script = cli.getOptionValue("script");
+ if (script != null) {
+ if (!(new File(script)).isFile())
+ throw new IllegalArgumentException(
+ "Value of -script option is invalid! " + script + " not found");
+ } else {
+ File scriptFile = new File(serverDir.getParentFile(), "bin/solr");
+ if (scriptFile.isFile()) {
+ script = scriptFile.getAbsolutePath();
+ } else {
+ scriptFile = new File(serverDir.getParentFile(), "bin/solr.cmd");
+ if (scriptFile.isFile()) {
+ script = scriptFile.getAbsolutePath();
+ } else {
+ throw new IllegalArgumentException(
+ "Cannot locate the bin/solr script! Please pass -script to this application.");
+ }
+ }
+ }
+
+ exampleDir =
+ (cli.hasOption("exampleDir"))
+ ? new File(cli.getOptionValue("exampleDir"))
+ : new File(serverDir.getParent(), "example");
+ if (!exampleDir.isDirectory())
+ throw new IllegalArgumentException(
+ "Value of -exampleDir option is invalid! "
+ + exampleDir.getAbsolutePath()
+ + " is not a directory!");
+
+ echoIfVerbose(
+ "Running with\nserverDir="
+ + serverDir.getAbsolutePath()
+ + ",\nexampleDir="
+ + exampleDir.getAbsolutePath()
+ + "\nscript="
+ + script,
+ cli);
+
+ String exampleType = cli.getOptionValue("example");
+ if ("cloud".equals(exampleType)) {
+ runCloudExample(cli);
+ } else if ("techproducts".equals(exampleType)
+ || "schemaless".equals(exampleType)
+ || "films".equals(exampleType)) {
+ runExample(cli, exampleType);
+ } else {
+ throw new IllegalArgumentException(
+ "Unsupported example "
+ + exampleType
+ + "! Please choose one of: cloud, schemaless, techproducts, or films");
+ }
+ }
+
+ protected void runExample(CommandLine cli, String exampleName) throws Exception {
+ File exDir = setupExampleDir(serverDir, exampleDir, exampleName);
+ String collectionName = "schemaless".equals(exampleName) ? "gettingstarted" : exampleName;
+ String configSet =
+ "techproducts".equals(exampleName) ? "sample_techproducts_configs" : "_default";
+
+ boolean isCloudMode = cli.hasOption('c');
+ String zkHost = cli.getOptionValue('z');
+ int port = Integer.parseInt(cli.getOptionValue('p', "8983"));
+ Map<String, Object> nodeStatus =
+ startSolr(new File(exDir, "solr"), isCloudMode, cli, port, zkHost, 30);
+
+ // invoke the CreateTool
+ File configsetsDir = new File(serverDir, "solr/configsets");
+
+ String solrUrl = (String) nodeStatus.get("baseUrl");
+
+ // safe check if core / collection already exists
+ boolean alreadyExists = false;
+ if (nodeStatus.get("cloud") != null) {
+ String collectionListUrl = solrUrl + "/admin/collections?action=list";
+ if (SolrCLI.safeCheckCollectionExists(collectionListUrl, collectionName)) {
+ alreadyExists = true;
+ echo(
+ "\nWARNING: Collection '"
+ + collectionName
+ + "' already exists!\nChecked collection existence using Collections API command:\n"
+ + collectionListUrl
+ + "\n");
+ }
+ } else {
+ String coreName = collectionName;
+ String coreStatusUrl = solrUrl + "/admin/cores?action=STATUS&core=" + coreName;
+ if (SolrCLI.safeCheckCoreExists(coreStatusUrl, coreName)) {
+ alreadyExists = true;
+ echo(
+ "\nWARNING: Core '"
+ + coreName
+ + "' already exists!\nChecked core existence using Core API command:\n"
+ + coreStatusUrl
+ + "\n");
+ }
+ }
+
+ if (!alreadyExists) {
+ String[] createArgs =
+ new String[]{
+ "-name", collectionName,
+ "-shards", "1",
+ "-replicationFactor", "1",
+ "-confname", collectionName,
+ "-confdir", configSet,
+ "-configsetsDir", configsetsDir.getAbsolutePath(),
+ "-solrUrl", solrUrl
+ };
+ CreateTool createTool = new CreateTool(stdout);
+ int createCode =
+ createTool.runTool(
+ SolrCLI.processCommandLineArgs(
+ createTool.getName(),
+ SolrCLI.joinCommonAndToolOptions(createTool.getOptions()),
+ createArgs));
+ if (createCode != 0)
+ throw new Exception(
+ "Failed to create "
+ + collectionName
+ + " using command: "
+ + Arrays.asList(createArgs));
+ }
+
+ if ("techproducts".equals(exampleName) && !alreadyExists) {
+
+ File exampledocsDir = new File(exampleDir, "exampledocs");
+ if (!exampledocsDir.isDirectory()) {
+ File readOnlyExampleDir = new File(serverDir.getParentFile(), "example");
+ if (readOnlyExampleDir.isDirectory()) {
+ exampledocsDir = new File(readOnlyExampleDir, "exampledocs");
+ }
+ }
+
+ if (exampledocsDir.isDirectory()) {
+ String updateUrl = String.format(Locale.ROOT, "%s/%s/update", solrUrl, collectionName);
+ echo("Indexing tech product example docs from " + exampledocsDir.getAbsolutePath());
+
+ String currentPropVal = System.getProperty("url");
+ System.setProperty("url", updateUrl);
+ SimplePostTool.main(new String[]{exampledocsDir.getAbsolutePath() + "/*.xml"});
+ if (currentPropVal != null) {
+ System.setProperty("url", currentPropVal); // reset
+ } else {
+ System.clearProperty("url");
+ }
+ } else {
+ echo(
+ "exampledocs directory not found, skipping indexing step for the techproducts example");
+ }
+ } else if ("films".equals(exampleName) && !alreadyExists) {
+ SolrClient solrClient = new HttpSolrClient.Builder(solrUrl).build();
+
+ echo("Adding dense vector field type to films schema \"_default\"");
+ try {
+ SolrCLI.postJsonToSolr(
+ solrClient,
+ "/" + collectionName + "/schema",
+ "{\n"
+ + " \"add-field-type\" : {\n"
+ + " \"name\":\"knn_vector_10\",\n"
+ + " \"class\":\"solr.DenseVectorField\",\n"
+ + " \"vectorDimension\":10,\n"
+ + " \"similarityFunction\":cosine\n"
+ + " \"knnAlgorithm\":hnsw\n"
+ + " }\n"
+ + " }");
+ } catch (Exception ex) {
+ throw new SolrException(SolrException.ErrorCode.SERVER_ERROR, ex);
+ }
+
+ echo(
+ "Adding name, initial_release_date, and film_vector fields to films schema \"_default\"");
+ try {
+ SolrCLI.postJsonToSolr(
+ solrClient,
+ "/" + collectionName + "/schema",
+ "{\n"
+ + " \"add-field\" : {\n"
+ + " \"name\":\"name\",\n"
+ + " \"type\":\"text_general\",\n"
+ + " \"multiValued\":false,\n"
+ + " \"stored\":true\n"
+ + " },\n"
+ + " \"add-field\" : {\n"
+ + " \"name\":\"initial_release_date\",\n"
+ + " \"type\":\"pdate\",\n"
+ + " \"stored\":true\n"
+ + " },\n"
+ + " \"add-field\" : {\n"
+ + " \"name\":\"film_vector\",\n"
+ + " \"type\":\"knn_vector_10\",\n"
+ + " \"indexed\":true\n"
+ + " \"stored\":true\n"
+ + " }\n"
+ + " }");
+ } catch (Exception ex) {
+ throw new SolrException(SolrException.ErrorCode.SERVER_ERROR, ex);
+ }
+
+ echo(
+ "Adding paramsets \"algo\" and \"algo_b\" to films configuration for relevancy tuning");
+ try {
+ SolrCLI.postJsonToSolr(
+ solrClient,
+ "/" + collectionName + "/config/params",
+ "{\n"
+ + " \"set\": {\n"
+ + " \"algo_a\":{\n"
+ + " \"defType\":\"dismax\",\n"
+ + " \"qf\":\"name\"\n"
+ + " }\n"
+ + " },\n"
+ + " \"set\": {\n"
+ + " \"algo_b\":{\n"
+ + " \"defType\":\"dismax\",\n"
+ + " \"qf\":\"name\",\n"
+ + " \"mm\":\"100%\"\n"
+ + " }\n"
+ + " }\n"
+ + " }\n");
+ } catch (Exception ex) {
+ throw new SolrException(SolrException.ErrorCode.SERVER_ERROR, ex);
+ }
+
+ File filmsJsonFile = new File(exampleDir, "films/films.json");
+ String updateUrl = String.format(Locale.ROOT, "%s/%s/update/json", solrUrl, collectionName);
+ echo("Indexing films example docs from " + filmsJsonFile.getAbsolutePath());
+ String currentPropVal = System.getProperty("url");
+ System.setProperty("url", updateUrl);
+ SimplePostTool.main(new String[]{filmsJsonFile.getAbsolutePath()});
+ if (currentPropVal != null) {
+ System.setProperty("url", currentPropVal); // reset
+ } else {
+ System.clearProperty("url");
+ }
+ }
+
+ echo(
+ "\nSolr "
+ + exampleName
+ + " example launched successfully. Direct your Web browser to "
+ + solrUrl
+ + " to visit the Solr Admin UI");
+ }
+
+ protected void runCloudExample(CommandLine cli) throws Exception {
+
+ boolean prompt = !cli.hasOption("noprompt");
+ int numNodes = 2;
+ int[] cloudPorts = new int[]{8983, 7574, 8984, 7575};
+ File cloudDir = new File(exampleDir, "cloud");
+ if (!cloudDir.isDirectory()) {
+ cloudDir.mkdir();
+ }
+
+ echo("\nWelcome to the SolrCloud example!\n");
+
+ Scanner readInput = prompt ? new Scanner(userInput, StandardCharsets.UTF_8.name()) : null;
+ if (prompt) {
+ echo(
+ "This interactive session will help you launch a SolrCloud cluster on your local workstation.");
+
+ // get the number of nodes to start
+ numNodes =
+ promptForInt(
+ readInput,
+ "To begin, how many Solr nodes would you like to run in your local cluster? (specify 1-4 nodes) [2]: ",
+ "a number",
+ numNodes,
+ 1,
+ 4);
+
+ echo("Ok, let's start up " + numNodes + " Solr nodes for your example SolrCloud cluster.");
+
+ // get the ports for each port
+ for (int n = 0; n < numNodes; n++) {
+ String promptMsg =
+ String.format(
+ Locale.ROOT, "Please enter the port for node%d [%d]: ", (n + 1), cloudPorts[n]);
+ int port = promptForPort(readInput, n + 1, promptMsg, cloudPorts[n]);
+ while (!isPortAvailable(port)) {
+ port =
+ promptForPort(
+ readInput,
+ n + 1,
+ "Oops! Looks like port "
+ + port
+ + " is already being used by another process. Please choose a different port.",
+ cloudPorts[n]);
+ }
+
+ cloudPorts[n] = port;
+ echoIfVerbose("Using port " + port + " for node " + (n + 1), cli);
+ }
+ } else {
+ echo("Starting up " + numNodes + " Solr nodes for your example SolrCloud cluster.\n");
+ }
+
+ // setup a unique solr.solr.home directory for each node
+ File node1Dir = setupExampleDir(serverDir, cloudDir, "node1");
+ for (int n = 2; n <= numNodes; n++) {
+ File nodeNDir = new File(cloudDir, "node" + n);
+ if (!nodeNDir.isDirectory()) {
+ echo("Cloning " + node1Dir.getAbsolutePath() + " into\n " + nodeNDir.getAbsolutePath());
+ FileUtils.copyDirectory(node1Dir, nodeNDir);
+ } else {
+ echo(nodeNDir.getAbsolutePath() + " already exists.");
+ }
+ }
+
+ // deal with extra args passed to the script to run the example
+ String zkHost = cli.getOptionValue('z');
+
+ // start the first node (most likely with embedded ZK)
+ Map<String, Object> nodeStatus =
+ startSolr(new File(node1Dir, "solr"), true, cli, cloudPorts[0], zkHost, 30);
+
+ if (zkHost == null) {
+ @SuppressWarnings("unchecked")
+ Map<String, Object> cloudStatus = (Map<String, Object>) nodeStatus.get("cloud");
+ if (cloudStatus != null) {
+ String zookeeper = (String) cloudStatus.get("ZooKeeper");
+ if (zookeeper != null) zkHost = zookeeper;
+ }
+ if (zkHost == null)
+ throw new Exception("Could not get the ZooKeeper connection string for node1!");
+ }
+
+ if (numNodes > 1) {
+ // start the other nodes
+ for (int n = 1; n < numNodes; n++)
+ startSolr(
+ new File(cloudDir, "node" + (n + 1) + "/solr"), true, cli, cloudPorts[n], zkHost, 30);
+ }
+
+ String solrUrl = (String) nodeStatus.get("baseUrl");
+ if (solrUrl.endsWith("/")) solrUrl = solrUrl.substring(0, solrUrl.length() - 1);
+
+ // wait until live nodes == numNodes
+ waitToSeeLiveNodes(10 /* max wait */, zkHost, numNodes);
+
+ // create the collection
+ String collectionName = createCloudExampleCollection(numNodes, readInput, prompt, solrUrl);
+
+ // update the config to enable soft auto-commit
+ echo("\nEnabling auto soft-commits with maxTime 3 secs using the Config API");
+ setCollectionConfigProperty(
+ solrUrl, collectionName, "updateHandler.autoSoftCommit.maxTime", "3000");
+
+ echo("\n\nSolrCloud example running, please visit: " + solrUrl + " \n");
+ }
+
+ protected void setCollectionConfigProperty(
+ String solrUrl, String collectionName, String propName, String propValue) {
+ ConfigTool configTool = new ConfigTool(stdout);
+ String[] configArgs =
+ new String[]{
+ "-collection",
+ collectionName,
+ "-property",
+ propName,
+ "-value",
+ propValue,
+ "-solrUrl",
+ solrUrl
+ };
+
+ // let's not fail if we get this far ... just report error and finish up
+ try {
+ configTool.runTool(
+ SolrCLI.processCommandLineArgs(
+ configTool.getName(),
+ SolrCLI.joinCommonAndToolOptions(configTool.getOptions()),
+ configArgs));
+ } catch (Exception exc) {
+ CLIO.err("Failed to update '" + propName + "' property due to: " + exc);
+ }
+ }
+
+ protected void waitToSeeLiveNodes(int maxWaitSecs, String zkHost, int numNodes) {
+ try (CloudSolrClient cloudClient =
+ new CloudSolrClient.Builder(Collections.singletonList(zkHost), Optional.empty())
+ .build()){
+
+ cloudClient.connect();
+ Set<String> liveNodes = cloudClient.getClusterState().getLiveNodes();
+ int numLiveNodes = (liveNodes != null) ? liveNodes.size() : 0;
+ long timeout =
+ System.nanoTime() + TimeUnit.NANOSECONDS.convert(maxWaitSecs, TimeUnit.SECONDS);
+ while (System.nanoTime() < timeout && numLiveNodes < numNodes) {
+ echo(
+ "\nWaiting up to "
+ + maxWaitSecs
+ + " seconds to see "
+ + (numNodes - numLiveNodes)
+ + " more nodes join the SolrCloud cluster ...");
+ try {
+ Thread.sleep(2000);
+ } catch (InterruptedException ie) {
+ Thread.interrupted();
+ }
+ liveNodes = cloudClient.getClusterState().getLiveNodes();
+ numLiveNodes = (liveNodes != null) ? liveNodes.size() : 0;
+ }
+ if (numLiveNodes < numNodes) {
+ echo(
+ "\nWARNING: Only "
+ + numLiveNodes
+ + " of "
+ + numNodes
+ + " are active in the cluster after "
+ + maxWaitSecs
+ + " seconds! Please check the solr.log for each node to look for errors.\n");
+ }
+ } catch (Exception exc) {
+ CLIO.err("Failed to see if " + numNodes + " joined the SolrCloud cluster due to: " + exc);
+ }
+ }
+
+ protected Map<String, Object> startSolr(
+ File solrHomeDir,
+ boolean cloudMode,
+ CommandLine cli,
+ int port,
+ String zkHost,
+ int maxWaitSecs)
+ throws Exception {
+
+ String extraArgs = readExtraArgs(cli.getArgs());
+
+ String host = cli.getOptionValue('h');
+ String memory = cli.getOptionValue('m');
+
+ String hostArg = (host != null && !"localhost".equals(host)) ? " -h " + host : "";
+ String zkHostArg = (zkHost != null) ? " -z " + zkHost : "";
+ String memArg = (memory != null) ? " -m " + memory : "";
+ String cloudModeArg = cloudMode ? "-cloud " : "";
+ String forceArg = cli.hasOption("force") ? " -force" : "";
+
+ String addlOpts = cli.getOptionValue('a');
+ String addlOptsArg = (addlOpts != null) ? " -a \"" + addlOpts + "\"" : "";
+
+ File cwd = new File(System.getProperty("user.dir"));
+ File binDir = (new File(script)).getParentFile();
+
+ boolean isWindows = (OS.isFamilyDOS() || OS.isFamilyWin9x() || OS.isFamilyWindows());
+ String callScript = (!isWindows && cwd.equals(binDir.getParentFile())) ? "bin/solr" : script;
+
+ String cwdPath = cwd.getAbsolutePath();
+ String solrHome = solrHomeDir.getAbsolutePath();
+
+ // don't display a huge path for solr home if it is relative to the cwd
+ if (!isWindows && cwdPath.length() > 1 && solrHome.startsWith(cwdPath))
+ solrHome = solrHome.substring(cwdPath.length() + 1);
+
+ String startCmd =
+ String.format(
+ Locale.ROOT,
+ "\"%s\" start %s -p %d -s \"%s\" %s %s %s %s %s %s",
+ callScript,
+ cloudModeArg,
+ port,
+ solrHome,
+ hostArg,
+ zkHostArg,
+ memArg,
+ forceArg,
+ extraArgs,
+ addlOptsArg);
+ startCmd = startCmd.replaceAll("\\s+", " ").trim(); // for pretty printing
+
+ echo("\nStarting up Solr on port " + port + " using command:");
+ echo(startCmd + "\n");
+
+ String solrUrl =
+ String.format(
+ Locale.ROOT, "%s://%s:%d/solr", urlScheme, (host != null ? host : "localhost"), port);
+
+ Map<String, Object> nodeStatus = checkPortConflict(solrUrl, solrHomeDir, port);
+ if (nodeStatus != null)
+ return nodeStatus; // the server they are trying to start is already running
+
+ int code = 0;
+ if (isWindows) {
+ // On Windows, the execution doesn't return, so we have to execute async
+ // and when calling the script, it seems to be inheriting the environment that launched this
+ // app, so we have to prune out env vars that may cause issues
+ Map<String, String> startEnv = new HashMap<>();
+ Map<String, String> procEnv = EnvironmentUtils.getProcEnvironment();
+ if (procEnv != null) {
+ for (Map.Entry<String, String> entry : procEnv.entrySet()) {
+ String envVar = entry.getKey();
+ String envVarVal = entry.getValue();
+ if (envVarVal != null && !"EXAMPLE".equals(envVar) && !envVar.startsWith("SOLR_")) {
+ startEnv.put(envVar, envVarVal);
+ }
+ }
+ }
+ DefaultExecuteResultHandler handler = new DefaultExecuteResultHandler();
+ executor.execute(org.apache.commons.exec.CommandLine.parse(startCmd), startEnv, handler);
+
+ // wait for execution.
+ try {
+ handler.waitFor(3000);
+ } catch (InterruptedException ie) {
+ // safe to ignore ...
+ Thread.interrupted();
+ }
+ if (handler.hasResult() && handler.getExitValue() != 0) {
+ throw new Exception(
+ "Failed to start Solr using command: "
+ + startCmd
+ + " Exception : "
+ + handler.getException());
+ }
+ } else {
+ try {
+ code = executor.execute(org.apache.commons.exec.CommandLine.parse(startCmd));
+ } catch (ExecuteException e) {
+ throw new Exception(
+ "Failed to start Solr using command: " + startCmd + " Exception : " + e);
+ }
+ }
+ if (code != 0) throw new Exception("Failed to start Solr using command: " + startCmd);
+
+ return getNodeStatus(solrUrl, maxWaitSecs);
+ }
+
+ protected Map<String, Object> checkPortConflict(
+ String solrUrl, File solrHomeDir, int port) {
+ // quickly check if the port is in use
+ if (isPortAvailable(port)) return null; // not in use ... try to start
+
+ Map<String, Object> nodeStatus = null;
+ try {
+ nodeStatus = (new StatusTool()).getStatus(solrUrl);
+ } catch (Exception ignore) {
+ /* just trying to determine if this example is already running. */
+ }
+
+ if (nodeStatus != null) {
+ String solr_home = (String) nodeStatus.get("solr_home");
+ if (solr_home != null) {
+ String solrHomePath = solrHomeDir.getAbsolutePath();
+ if (!solrHomePath.endsWith("/")) solrHomePath += "/";
+ if (!solr_home.endsWith("/")) solr_home += "/";
+
+ if (solrHomePath.equals(solr_home)) {
+ CharArr arr = new CharArr();
+ new JSONWriter(arr, 2).write(nodeStatus);
+ echo(
+ "Solr is already setup and running on port "
+ + port
+ + " with status:\n"
+ + arr.toString());
+ echo(
+ "\nIf this is not the example node you are trying to start, please choose a different port.");
+ nodeStatus.put("baseUrl", solrUrl);
+ return nodeStatus;
+ }
+ }
+ }
+
+ throw new IllegalStateException(
+ "Port " + port + " is already being used by another process.");
+ }
+
+ protected String readExtraArgs(String[] extraArgsArr) {
+ String extraArgs = "";
+ if (extraArgsArr != null && extraArgsArr.length > 0) {
+ StringBuilder sb = new StringBuilder();
+ int app = 0;
+ for (int e = 0; e < extraArgsArr.length; e++) {
+ String arg = extraArgsArr[e];
+ if ("e".equals(arg) || "example".equals(arg)) {
+ e++; // skip over the example arg
+ continue;
+ }
+
+ if (app > 0) sb.append(" ");
+ sb.append(arg);
+ ++app;
+ }
+ extraArgs = sb.toString().trim();
+ }
+ return extraArgs;
+ }
+
+ protected String createCloudExampleCollection(
+ int numNodes, Scanner readInput, boolean prompt, String solrUrl) throws Exception {
+ // yay! numNodes SolrCloud nodes running
+ int numShards = 2;
+ int replicationFactor = 2;
+ String cloudConfig = "_default";
+ String collectionName = "gettingstarted";
+
+ File configsetsDir = new File(serverDir, "solr/configsets");
+ String collectionListUrl = solrUrl + "/admin/collections?action=list";
+
+ if (prompt) {
+ echo(
+ "\nNow let's create a new collection for indexing documents in your "
+ + numNodes
+ + "-node cluster.");
+
+ while (true) {
+ collectionName =
+ prompt(
+ readInput,
+ "Please provide a name for your new collection: [" + collectionName + "] ",
+ collectionName);
+
+ // Test for existence and then prompt to either create another collection or skip the creation step
+ if (SolrCLI.safeCheckCollectionExists(collectionListUrl, collectionName)) {
+ echo("\nCollection '" + collectionName + "' already exists!");
+ int oneOrTwo =
+ promptForInt(
+ readInput,
+ "Do you want to re-use the existing collection or create a new one? Enter 1 to reuse, 2 to create new [1]: ",
+ "a 1 or 2",
+ 1,
+ 1,
+ 2);
+ if (oneOrTwo == 1) {
+ return collectionName;
+ } else {
+ continue;
+ }
+ } else {
+ break; // user selected a collection that doesn't exist ... proceed on
+ }
+ }
+
+ numShards =
+ promptForInt(
+ readInput,
+ "How many shards would you like to split " + collectionName + " into? [2]",
+ "a shard count",
+ 2,
+ 1,
+ 4);
+
+ replicationFactor =
+ promptForInt(
+ readInput,
+ "How many replicas per shard would you like to create? [2] ",
+ "a replication factor",
+ 2,
+ 1,
+ 4);
+
+ echo(
+ "Please choose a configuration for the "
+ + collectionName
+ + " collection, available options are:");
+ String validConfigs = "_default or sample_techproducts_configs [" + cloudConfig + "] ";
+ cloudConfig = prompt(readInput, validConfigs, cloudConfig);
+
+ // validate the cloudConfig name
+ while (!isValidConfig(configsetsDir, cloudConfig)) {
+ echo(
+ cloudConfig
+ + " is not a valid configuration directory! Please choose a configuration for the "
+ + collectionName
+ + " collection, available options are:");
+ cloudConfig = prompt(readInput, validConfigs, cloudConfig);
+ }
+ } else {
+ // must verify if default collection exists
+ if (SolrCLI.safeCheckCollectionExists(collectionListUrl, collectionName)) {
+ echo(
+ "\nCollection '"
+ + collectionName
+ + "' already exists! Skipping collection creation step.");
+ return collectionName;
+ }
+ }
+
+ // invoke the CreateCollectionTool
+ String[] createArgs =
+ new String[]{
+ "-name", collectionName,
+ "-shards", String.valueOf(numShards),
+ "-replicationFactor", String.valueOf(replicationFactor),
+ "-confname", collectionName,
+ "-confdir", cloudConfig,
+ "-configsetsDir", configsetsDir.getAbsolutePath(),
+ "-solrUrl", solrUrl
+ };
+
+ CreateCollectionTool createCollectionTool = new CreateCollectionTool(stdout);
+ int createCode =
+ createCollectionTool.runTool(
+ SolrCLI.processCommandLineArgs(
+ createCollectionTool.getName(),
+ SolrCLI.joinCommonAndToolOptions(createCollectionTool.getOptions()),
+ createArgs));
+
+ if (createCode != 0)
+ throw new Exception(
+ "Failed to create collection using command: " + Arrays.asList(createArgs));
+
+ return collectionName;
+ }
+
+ protected boolean isValidConfig(File configsetsDir, String config) {
+ File configDir = new File(configsetsDir, config);
+ if (configDir.isDirectory()) return true;
+
+ // not a built-in configset ... maybe it's a custom directory?
+ configDir = new File(config);
+ return configDir.isDirectory();
+ }
+
+ protected Map<String, Object> getNodeStatus(String solrUrl, int maxWaitSecs) throws Exception {
+ StatusTool statusTool = new StatusTool();
+ if (verbose) echo("\nChecking status of Solr at " + solrUrl + " ...");
+
+ URL solrURL = new URL(solrUrl);
+ Map<String, Object> nodeStatus = statusTool.waitToSeeSolrUp(solrUrl, maxWaitSecs);
+ nodeStatus.put("baseUrl", solrUrl);
+ CharArr arr = new CharArr();
+ new JSONWriter(arr, 2).write(nodeStatus);
+ String mode = (nodeStatus.get("cloud") != null) ? "cloud" : "standalone";
+ if (verbose)
+ echo(
+ "\nSolr is running on "
+ + solrURL.getPort()
+ + " in "
+ + mode
+ + " mode with status:\n"
+ + arr);
+
+ return nodeStatus;
+ }
+
+ protected File setupExampleDir(File serverDir, File exampleParentDir, String dirName)
+ throws IOException {
+ File solrXml = new File(serverDir, "solr/solr.xml");
+ if (!solrXml.isFile())
+ throw new IllegalArgumentException(
+ "Value of -serverDir option is invalid! " + solrXml.getAbsolutePath() + " not found!");
+
+ File zooCfg = new File(serverDir, "solr/zoo.cfg");
+ if (!zooCfg.isFile())
+ throw new IllegalArgumentException(
+ "Value of -serverDir option is invalid! " + zooCfg.getAbsolutePath() + " not found!");
+
+ File solrHomeDir = new File(exampleParentDir, dirName + "/solr");
Review Comment:
<picture><img alt="5% of developers fix this issue" src="https://lift.sonatype.com/api/commentimage/fixrate/5/display.svg"></picture>
<b>*[PATH_TRAVERSAL_IN](https://find-sec-bugs.github.io/bugs.htm#PATH_TRAVERSAL_IN):</b>* This API (java/io/File.<init>(Ljava/io/File;Ljava/lang/String;)V) reads a file whose location might be specified by user input
ââ <b>24 similar findings have been found in this PR</b>
<details><summary>đ Expand here to view all instances of this finding</summary><br/>
<div align=\"center\">
| **File Path** | **Line Number** |
| ------------- | ------------- |
| solr/core/src/java/org/apache/solr/util/cli/AssertTool.java | [275](https://github.com/apache/solr/blob/1e4ea507fafe99971a1ff388d369474a6db74d18/solr/core/src/java/org/apache/solr/util/cli/AssertTool.java#L275) |
| solr/core/src/java/org/apache/solr/util/cli/AuthTool.java | [432](https://github.com/apache/solr/blob/1e4ea507fafe99971a1ff388d369474a6db74d18/solr/core/src/java/org/apache/solr/util/cli/AuthTool.java#L432) |
| solr/core/src/java/org/apache/solr/util/cli/RunExampleTool.java | [201](https://github.com/apache/solr/blob/1e4ea507fafe99971a1ff388d369474a6db74d18/solr/core/src/java/org/apache/solr/util/cli/RunExampleTool.java#L201) |
| solr/core/src/java/org/apache/solr/util/cli/CreateCoreTool.java | [71](https://github.com/apache/solr/blob/1e4ea507fafe99971a1ff388d369474a6db74d18/solr/core/src/java/org/apache/solr/util/cli/CreateCoreTool.java#L71) |
| solr/core/src/java/org/apache/solr/util/cli/AuthTool.java | [440](https://github.com/apache/solr/blob/1e4ea507fafe99971a1ff388d369474a6db74d18/solr/core/src/java/org/apache/solr/util/cli/AuthTool.java#L440) |
| solr/core/src/java/org/apache/solr/util/cli/ConfigSetDownloadTool.java | [73](https://github.com/apache/solr/blob/1e4ea507fafe99971a1ff388d369474a6db74d18/solr/core/src/java/org/apache/solr/util/cli/ConfigSetDownloadTool.java#L73) |
| solr/core/src/java/org/apache/solr/util/cli/CreateCoreTool.java | [79](https://github.com/apache/solr/blob/1e4ea507fafe99971a1ff388d369474a6db74d18/solr/core/src/java/org/apache/solr/util/cli/CreateCoreTool.java#L79) |
| solr/core/src/java/org/apache/solr/util/cli/ConfigSetDownloadTool.java | [69](https://github.com/apache/solr/blob/1e4ea507fafe99971a1ff388d369474a6db74d18/solr/core/src/java/org/apache/solr/util/cli/ConfigSetDownloadTool.java#L69) |
| solr/core/src/java/org/apache/solr/util/cli/RunExampleTool.java | [481](https://github.com/apache/solr/blob/1e4ea507fafe99971a1ff388d369474a6db74d18/solr/core/src/java/org/apache/solr/util/cli/RunExampleTool.java#L481) |
| solr/core/src/java/org/apache/solr/util/cli/AuthTool.java | [235](https://github.com/apache/solr/blob/1e4ea507fafe99971a1ff388d369474a6db74d18/solr/core/src/java/org/apache/solr/util/cli/AuthTool.java#L235) |
<p> Showing <b>10</b> of <b> 24 </b> findings. <a href="https://lift.sonatype.com/results/github.com/apache/solr/01GW023GCS4W08RFDKE0X17MZM?t=FindSecBugs|PATH_TRAVERSAL_IN" target="_blank">Visit the Lift Web Console</a> to see all.</p></div></details>
---
<details><summary>âšī¸ Expand to see all <b>@sonatype-lift</b> commands</summary>
You can reply with the following commands. For example, reply with ***@sonatype-lift ignoreall*** to leave out all findings.
| **Command** | **Usage** |
| ------------- | ------------- |
| `@sonatype-lift ignore` | Leave out the above finding from this PR |
| `@sonatype-lift ignoreall` | Leave out all the existing findings from this PR |
| `@sonatype-lift exclude <file\|issue\|path\|tool>` | Exclude specified `file\|issue\|path\|tool` from Lift findings by updating your config.toml file |
**Note:** When talking to LiftBot, you need to **refresh** the page to see its response.
<sub>[Click here](https://github.com/apps/sonatype-lift/installations/new) to add LiftBot to another repo.</sub></details>
---
<b>Help us improve LIFT! (<i>Sonatype LiftBot external survey</i>)</b>
Was this a good recommendation for you? <sub><small>Answering this survey will not impact your Lift settings.</small></sub>
[â[đ Not relevant](https://www.sonatype.com/lift-comment-rating?comment=444297187&lift_comment_rating=1)â] - [â[đ Won't fix](https://www.sonatype.com/lift-comment-rating?comment=444297187&lift_comment_rating=2)â] - [â[đ Not critical, will fix](https://www.sonatype.com/lift-comment-rating?comment=444297187&lift_comment_rating=3)â] - [â[đ Critical, will fix](https://www.sonatype.com/lift-comment-rating?comment=444297187&lift_comment_rating=4)â] - [â[đ Critical, fixing now](https://www.sonatype.com/lift-comment-rating?comment=444297187&lift_comment_rating=5)â]
##########
solr/core/src/java/org/apache/solr/util/cli/AssertTool.java:
##########
@@ -0,0 +1,362 @@
+package org.apache.solr.util.cli;
+
+import org.apache.commons.cli.CommandLine;
+import org.apache.commons.cli.HelpFormatter;
+import org.apache.commons.cli.Option;
+import org.apache.solr.client.solrj.SolrClient;
+import org.apache.solr.client.solrj.SolrRequest;
+import org.apache.solr.client.solrj.impl.HttpSolrClient;
+import org.apache.solr.client.solrj.request.CollectionAdminRequest;
+import org.apache.solr.client.solrj.response.CollectionAdminResponse;
+import org.apache.solr.common.SolrException;
+import org.apache.solr.util.CLIO;
+import org.apache.solr.util.SolrCLI;
+
+import java.io.IOException;
+import java.io.PrintStream;
+import java.nio.file.Files;
+import java.nio.file.Path;
+import java.nio.file.Paths;
+import java.nio.file.attribute.FileOwnerAttributeView;
+import java.util.Optional;
+import java.util.concurrent.TimeUnit;
+
+/**
+ * Asserts various conditions and exists with error code if fails, else continues with no output
+ */
+public class AssertTool extends ToolBase {
+
+ private static String message = null;
+ private static boolean useExitCode = false;
+ private static Optional<Long> timeoutMs = Optional.empty();
+
+ public AssertTool() {
+ this(CLIO.getOutStream());
+ }
+
+ public AssertTool(PrintStream stdout) {
+ super(stdout);
+ }
+
+ @Override
+ public String getName() {
+ return "assert";
+ }
+
+ @Override
+ public Option[] getOptions() {
+ return new Option[]{
+ Option.builder("R")
+ .desc("Asserts that we are NOT the root user.")
+ .longOpt("not-root")
+ .build(),
+ Option.builder("r").desc("Asserts that we are the root user.").longOpt("root").build(),
+ Option.builder("S")
+ .desc("Asserts that Solr is NOT running on a certain URL. Default timeout is 1000ms.")
+ .longOpt("not-started")
+ .hasArg(true)
+ .argName("url")
+ .build(),
+ Option.builder("s")
+ .desc("Asserts that Solr is running on a certain URL. Default timeout is 1000ms.")
+ .longOpt("started")
+ .hasArg(true)
+ .argName("url")
+ .build(),
+ Option.builder("u")
+ .desc("Asserts that we run as same user that owns <directory>.")
+ .longOpt("same-user")
+ .hasArg(true)
+ .argName("directory")
+ .build(),
+ Option.builder("x")
+ .desc("Asserts that directory <directory> exists.")
+ .longOpt("exists")
+ .hasArg(true)
+ .argName("directory")
+ .build(),
+ Option.builder("X")
+ .desc("Asserts that directory <directory> does NOT exist.")
+ .longOpt("not-exists")
+ .hasArg(true)
+ .argName("directory")
+ .build(),
+ Option.builder("c")
+ .desc(
+ "Asserts that Solr is running in cloud mode. Also fails if Solr not running. URL should be for root Solr path.")
+ .longOpt("cloud")
+ .hasArg(true)
+ .argName("url")
+ .build(),
+ Option.builder("C")
+ .desc(
+ "Asserts that Solr is not running in cloud mode. Also fails if Solr not running. URL should be for root Solr path.")
+ .longOpt("not-cloud")
+ .hasArg(true)
+ .argName("url")
+ .build(),
+ Option.builder("m")
+ .desc("Exception message to be used in place of the default error message.")
+ .longOpt("message")
+ .hasArg(true)
+ .argName("message")
+ .build(),
+ Option.builder("t")
+ .desc("Timeout in ms for commands supporting a timeout.")
+ .longOpt("timeout")
+ .hasArg(true)
+ .type(Long.class)
+ .argName("ms")
+ .build(),
+ Option.builder("e")
+ .desc("Return an exit code instead of printing error message on assert fail.")
+ .longOpt("exitcode")
+ .build()
+ };
+ }
+
+ @Override
+ public int runTool(CommandLine cli) throws Exception {
+ verbose = cli.hasOption(SolrCLI.OPTION_VERBOSE.getOpt());
+
+ int toolExitStatus;
+ try {
+ toolExitStatus = runAssert(cli);
+ } catch (Exception exc) {
+ // since this is a CLI, spare the user the stacktrace
+ String excMsg = exc.getMessage();
+ if (excMsg != null) {
+ if (verbose) {
+ CLIO.err("\nERROR: " + exc + "\n");
+ } else {
+ CLIO.err("\nERROR: " + excMsg + "\n");
+ }
+ toolExitStatus = 100; // Exit >= 100 means error, else means number of tests that failed
+ } else {
+ throw exc;
+ }
+ }
+ return toolExitStatus;
+ }
+
+ @Override
+ public void runImpl(CommandLine cli) throws Exception {
+ runAssert(cli);
+ }
+
+ /**
+ * Custom run method which may return exit code
+ *
+ * @param cli the command line object
+ * @return 0 on success, or a number corresponding to number of tests that failed
+ * @throws Exception if a tool failed, e.g. authentication failure
+ */
+ protected int runAssert(CommandLine cli) throws Exception {
+ if (cli.getOptions().length == 0 || cli.getArgs().length > 0 || cli.hasOption("h")) {
+ new HelpFormatter()
+ .printHelp(
+ "bin/solr assert [-m <message>] [-e] [-rR] [-s <url>] [-S <url>] [-c <url>] [-C <url>] [-u <dir>] [-x <dir>] [-X <dir>]",
+ SolrCLI.getToolOptions(this));
+ return 1;
+ }
+ if (cli.hasOption("m")) {
+ message = cli.getOptionValue("m");
+ }
+ if (cli.hasOption("t")) {
+ timeoutMs = Optional.of(Long.parseLong(cli.getOptionValue("t")));
+ }
+ if (cli.hasOption("e")) {
+ useExitCode = true;
+ }
+
+ int ret = 0;
+ if (cli.hasOption("r")) {
+ ret += assertRootUser();
+ }
+ if (cli.hasOption("R")) {
+ ret += assertNotRootUser();
+ }
+ if (cli.hasOption("x")) {
+ ret += assertFileExists(cli.getOptionValue("x"));
+ }
+ if (cli.hasOption("X")) {
+ ret += assertFileNotExists(cli.getOptionValue("X"));
+ }
+ if (cli.hasOption("u")) {
+ ret += sameUser(cli.getOptionValue("u"));
+ }
+ if (cli.hasOption("s")) {
+ ret += assertSolrRunning(cli.getOptionValue("s"));
+ }
+ if (cli.hasOption("S")) {
+ ret += assertSolrNotRunning(cli.getOptionValue("S"));
+ }
+ if (cli.hasOption("c")) {
+ ret += assertSolrRunningInCloudMode(cli.getOptionValue("c"));
+ }
+ if (cli.hasOption("C")) {
+ ret += assertSolrNotRunningInCloudMode(cli.getOptionValue("C"));
+ }
+ return ret;
+ }
+
+ public static int assertSolrRunning(String url) throws Exception {
+ StatusTool status = new StatusTool();
+ try {
+ status.waitToSeeSolrUp(url, timeoutMs.orElse(1000L).intValue() / 1000);
+ } catch (Exception se) {
+ if (SolrCLI.exceptionIsAuthRelated(se)) {
+ throw se;
+ }
+ return exitOrException(
+ "Solr is not running on url " + url + " after " + timeoutMs.orElse(1000L) / 1000 + "s");
+ }
+ return 0;
+ }
+
+ public static int assertSolrNotRunning(String url) throws Exception {
+ StatusTool status = new StatusTool();
+ long timeout =
+ System.nanoTime()
+ + TimeUnit.NANOSECONDS.convert(timeoutMs.orElse(1000L), TimeUnit.MILLISECONDS);
+ try {
+ SolrCLI.attemptHttpHead(url, SolrCLI.getHttpClient());
+ } catch (SolrException se) {
+ throw se; // Auth error
+ } catch (IOException e) {
+ SolrCLI.log.debug("Opening connection to {} failed, Solr does not seem to be running", url, e);
+ return 0;
+ }
+ while (System.nanoTime() < timeout) {
+ try {
+ status.waitToSeeSolrUp(url, 1);
+ try {
+ SolrCLI.log.debug("Solr still up. Waiting before trying again to see if it was stopped");
+ Thread.sleep(1000L);
+ } catch (InterruptedException interrupted) {
+ timeout = 0; // stop looping
+ }
+ } catch (Exception se) {
+ if (SolrCLI.exceptionIsAuthRelated(se)) {
+ throw se;
+ }
+ return exitOrException(se.getMessage());
+ }
+ }
+ return exitOrException(
+ "Solr is still running at " + url + " after " + timeoutMs.orElse(1000L) / 1000 + "s");
+ }
+
+ public static int assertSolrRunningInCloudMode(String url) throws Exception {
+ if (!isSolrRunningOn(url)) {
+ return exitOrException(
+ "Solr is not running on url " + url + " after " + timeoutMs.orElse(1000L) / 1000 + "s");
+ }
+
+ if (!runningSolrIsCloud(url)) {
+ return exitOrException("Solr is not running in cloud mode on " + url);
+ }
+ return 0;
+ }
+
+ public static int assertSolrNotRunningInCloudMode(String url) throws Exception {
+ if (!isSolrRunningOn(url)) {
+ return exitOrException(
+ "Solr is not running on url " + url + " after " + timeoutMs.orElse(1000L) / 1000 + "s");
+ }
+
+ if (runningSolrIsCloud(url)) {
+ return exitOrException("Solr is not running in standalone mode on " + url);
+ }
+ return 0;
+ }
+
+ public static int sameUser(String directory) throws Exception {
+ if (Files.exists(Paths.get(directory))) {
+ String userForDir = userForDir(Paths.get(directory));
+ if (!currentUser().equals(userForDir)) {
+ return exitOrException("Must run as user " + userForDir + ". We are " + currentUser());
+ }
+ } else {
+ return exitOrException("Directory " + directory + " does not exist.");
+ }
+ return 0;
+ }
+
+ public static int assertFileExists(String directory) throws Exception {
+ if (!Files.exists(Paths.get(directory))) {
+ return exitOrException("Directory " + directory + " does not exist.");
+ }
+ return 0;
+ }
+
+ public static int assertFileNotExists(String directory) throws Exception {
+ if (Files.exists(Paths.get(directory))) {
+ return exitOrException("Directory " + directory + " should not exist.");
+ }
+ return 0;
+ }
+
+ public static int assertRootUser() throws Exception {
+ if (!currentUser().equals("root")) {
+ return exitOrException("Must run as root user");
+ }
+ return 0;
+ }
+
+ public static int assertNotRootUser() throws Exception {
+ if (currentUser().equals("root")) {
Review Comment:
<picture><img alt="15% of developers fix this issue" src="https://lift.sonatype.com/api/commentimage/fixrate/15/display.svg"></picture>
<b>*NULL_DEREFERENCE:</b>* object returned by `currentUser()` could be null and is dereferenced at line 308.
ââ <b>4 similar findings have been found in this PR</b>
<details><summary>đ Expand here to view all instances of this finding</summary><br/>
<div align=\"center\">
| **File Path** | **Line Number** |
| ------------- | ------------- |
| solr/core/src/java/org/apache/solr/util/cli/AssertTool.java | [277](https://github.com/apache/solr/blob/1e4ea507fafe99971a1ff388d369474a6db74d18/solr/core/src/java/org/apache/solr/util/cli/AssertTool.java#L277) |
| solr/core/src/java/org/apache/solr/util/cli/StatusTool.java | [141](https://github.com/apache/solr/blob/1e4ea507fafe99971a1ff388d369474a6db74d18/solr/core/src/java/org/apache/solr/util/cli/StatusTool.java#L141) |
| solr/core/src/java/org/apache/solr/util/cli/StatusTool.java | [169](https://github.com/apache/solr/blob/1e4ea507fafe99971a1ff388d369474a6db74d18/solr/core/src/java/org/apache/solr/util/cli/StatusTool.java#L169) |
| solr/core/src/java/org/apache/solr/util/cli/AssertTool.java | [301](https://github.com/apache/solr/blob/1e4ea507fafe99971a1ff388d369474a6db74d18/solr/core/src/java/org/apache/solr/util/cli/AssertTool.java#L301) |
<p><a href="https://lift.sonatype.com/results/github.com/apache/solr/01GW023GCS4W08RFDKE0X17MZM?t=Infer|NULL_DEREFERENCE" target="_blank">Visit the Lift Web Console</a> to find more details in your report.</p></div></details>
---
<details><summary>âšī¸ Expand to see all <b>@sonatype-lift</b> commands</summary>
You can reply with the following commands. For example, reply with ***@sonatype-lift ignoreall*** to leave out all findings.
| **Command** | **Usage** |
| ------------- | ------------- |
| `@sonatype-lift ignore` | Leave out the above finding from this PR |
| `@sonatype-lift ignoreall` | Leave out all the existing findings from this PR |
| `@sonatype-lift exclude <file\|issue\|path\|tool>` | Exclude specified `file\|issue\|path\|tool` from Lift findings by updating your config.toml file |
**Note:** When talking to LiftBot, you need to **refresh** the page to see its response.
<sub>[Click here](https://github.com/apps/sonatype-lift/installations/new) to add LiftBot to another repo.</sub></details>
---
<b>Help us improve LIFT! (<i>Sonatype LiftBot external survey</i>)</b>
Was this a good recommendation for you? <sub><small>Answering this survey will not impact your Lift settings.</small></sub>
[â[đ Not relevant](https://www.sonatype.com/lift-comment-rating?comment=444298427&lift_comment_rating=1)â] - [â[đ Won't fix](https://www.sonatype.com/lift-comment-rating?comment=444298427&lift_comment_rating=2)â] - [â[đ Not critical, will fix](https://www.sonatype.com/lift-comment-rating?comment=444298427&lift_comment_rating=3)â] - [â[đ Critical, will fix](https://www.sonatype.com/lift-comment-rating?comment=444298427&lift_comment_rating=4)â] - [â[đ Critical, fixing now](https://www.sonatype.com/lift-comment-rating?comment=444298427&lift_comment_rating=5)â]
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@solr.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@solr.apache.org
For additional commands, e-mail: issues-help@solr.apache.org