Ticket #546 (accepted enhancement)
Solr-plugin to support (password protected) SSL/TLS
| Reported by: | anonymous | Owned by: | Frans |
|---|---|---|---|
| Priority: | minor | Milestone: | MediaMosa X.X |
| Component: | Core | Version: | |
| Keywords: | Cc: | ||
| MoSCoW: | none | Estimated time after impact analysis: | |
| Related to project: | none | Tested: | no |
| Accepted: | no | Estimated Hours: |
Description
See forum post
http://mediamosa.org/content/mediamosa-solr-module-strips-https-url
Eg.
https://mm-solr-test01.uio.no/solr
becomes
http://mm-solr-test01.uio.no:8983/solr/
whereas it should be interpreted as
https://mm-solr-test01.uio.no:443/solr
Fix:
{{{diff -r a33bbefa1781 sites/all/modules/mediamosa_solr/mediamosa_solr_apache_solr_service.class.inc
--- a/sites/all/modules/mediamosa_solr/mediamosa_solr_apache_solr_service.class.inc Thu Oct 13 11:49:14 2011 +0200
+++ b/sites/all/modules/mediamosa_solr/mediamosa_solr_apache_solr_service.class.inc Wed Oct 26 15:34:28 2011 +0200
@@ -128,8 +128,8 @@
'path' => self::MEDIAMOSA_SOLR_DEFAULT_PATH,
);
- // Solr only uses host, port and path.
- return ' http://' . host? . ':' . port? . '/' . trim(path?, '/') . '/';
+ // Solr only uses scheme, host, port and path.
+ return scheme? . '://' . host? . ':' . port? . '/' . trim(path?, '/') . '/';
}
}}}
Additionally, the solr-plugin should support password protected solr.
