[
https://issues.apache.org/jira/browse/SOLR-3854?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13894090#comment-13894090
]
Steve Davids commented on SOLR-3854:
------------------------------------
Thanks, let me know if you need a hand. I was also thinking that the following
test should be added to verify the scheme:
{code}
private void testBaseUrlHttpsScheme() {
List<Replica> replicas = getZkReplicas();
assertFalse("No replicas found in ZooKeeper", replicas.isEmpty());
for(Replica replica : replicas) {
String baseUrl = (String) replica.get(ZkStateReader.BASE_URL_PROP);
assertTrue(baseUrl + " didn't begin with a https url scheme",
StringUtils.startsWith(baseUrl, "https://"));
try {
URL url = new URL(baseUrl);
assertNotNull("No path can be found for " + replica.getNodeName(),
url.getPath());
} catch (Exception ex) {
fail(replica.getNodeName() + " failed to build a proper URL [" +
baseUrl + "]");
}
}
}
protected List<Replica> getZkReplicas() {
List<Replica> replicas = new ArrayList<Replica>();
ClusterState clusterState =
cloudClient.getZkStateReader().getClusterState();
for(String collection : clusterState.getCollections()) {
for(Slice slice : clusterState.getSlicesMap(collection).values()) {
replicas.addAll(slice.getReplicas());
}
}
return replicas;
}
{code}
> SolrCloud does not work with https
> ----------------------------------
>
> Key: SOLR-3854
> URL: https://issues.apache.org/jira/browse/SOLR-3854
> Project: Solr
> Issue Type: Bug
> Reporter: Sami Siren
> Assignee: Mark Miller
> Fix For: 5.0, 4.7
>
> Attachments: SOLR-3854.patch, SOLR-3854.patch, SOLR-3854.patch,
> SOLR-3854.patch, SOLR-3854.patch, SOLR-3854.patch, SOLR-3854.patch,
> SOLR-3854.patch
>
>
> There are a few places in current codebase that assume http is used. This
> prevents using https when running solr in cloud mode.
--
This message was sent by Atlassian JIRA
(v6.1.5#6160)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]