Problem/Motivation

Recently, opensolr made it mandatory for indexes to have Basic Auth enabled, therefor the default connector is not needed anymore.

Proposed resolution

Remove the default connector from the codebase and find a way to not affect existing usages although it's impossible to use the opensolr connector without basic auth. But just in case an update to change the connector would be a good idea.

Comments

bbu23 created an issue. See original summary.

  • bbu23 committed 59d3b8a on 1.0.x
    Issue #3257909. Mark the default connector as deprecated and the basic...
gagarine’s picture

Then what is the use of the credential config in admin/config/search/search-api/opensolr when using basic auth?

bbu23’s picture

The Opensolr configuration form is required for API calls between Drupal and Opensolr. Without it, Drupal would not be able to list the available indexes or do similar API functionality.
https://opensolr.com/faq/view/api-index-management

On the other hand, the Basic Auth at index level adds an extra layer of security when searching for content.

gagarine’s picture

Ok I understand. I end up not using the module (one less dependency!).

It was quiet easy to not use the module at the end and copy/past the needed configuration (server adresse and core name).
But I can see the value if you have to manage a lot of config. Thanks for the work!

bbu23’s picture

Sure, people are free to go for any of the two modules, depending on their own comfort and usage.

bbu23’s picture

Version: 8.x-1.x-dev » 2.2.x-dev

  • bbu23 committed f547ed8d on 2.2.x
    Issue #3257909: Remove deprecated opensolr connector and replace...
bbu23’s picture

Status: Active » Fixed

Fixed and will be available starting with version 2.2.0.
One must run the updates when downloading the new version and export the changes.
All potential existing servers that are still using the deprecated opensolr connector will be converted to basic_auth_opensolr.

  • bbu23 committed bdef6e80 on 2.2.x
    Issue #3257909: Add forgotten use statement for NestedArray.
    

  • bbu23 committed 3f4c8c1d on 2.2.x
    Issue #3257909: Update README.md file.
    

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.