Uploaded image for project: 'Solr'
  1. Solr
  2. SOLR-8056

Add a Solrj client constructor to accept custom SolrParams

Attach filesAttach ScreenshotAdd voteVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Open
    • Minor
    • Resolution: Unresolved
    • 4.10.3, 6.0
    • None
    • SolrJ
    • None

    Description

      Solrj users can customize the Http client by overriding HttpClientConfigurer class (e.g. Krb5HttpClientConfigurer). But Solrj does not allow this customization at an individual SolrClient level. e.g. HttpSolrClient does not have a constructor which accepts default SolrParams which would be passed down to HttpClientUtil while creating an internal client.

      I observed this while working on supporting Basic authentication with Solr (4.10.3) version. As a work-around, I had to use external Http client support in Solr. e.g.

      ----------------------------BEFORE-----------------------------
      CloudSolrServer cloudSolrServer = new CloudSolrServer(zkEnsemble);

      -----------------------------AFTER-------------------------------
      ModifiableSolrParams solrParams = new ModifiableSolrParams();
      solrParams.set(HttpClientUtil.PROP_BASIC_AUTH_USER, getLdapUserName());
      solrParams.set(HttpClientUtil.PROP_BASIC_AUTH_PASS, getLdapUserPasswd());

      DefaultHttpClient client = (DefaultHttpClient) HttpClientUtil.createClient(solrParams);
      cloudSolrServer = new CloudSolrServer(zkEnsemble, new LBHttpSolrServer(client));
      --------------------------------------------------------------------------------------------------

      I think it would be great necessary constructors in the SolrClient implementations so that users can pass custom properties during initialization.

      Attachments

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            Unassigned Unassigned
            hgadre Hrishikesh Gadre

            Dates

              Created:
              Updated:

              Slack

                Issue deployment