Ticket #546 (accepted enhancement)

Opened 18 months ago

Last modified 17 months ago

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.

Change History

Changed 17 months ago by Frans

  • owner set to Frans
  • status set to accepted
Note: See TracTickets for help on using tickets.