Problem/Motivation

For developer purposes it would be nice to have an ability disable SSl verification for open search server.

Proposed resolution

Add a new checkbox to the connector configuration form in order to enable/disable SSL verification

CommentFileSizeAuthor
#6 search_api_opensearch-ssl-verification-option-3381020-6.patch5.84 KBad0z

Issue fork search_api_opensearch-3381020

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

afi13 created an issue. See original summary.

afi13 opened merge request !37

afi13’s picture

afi13 commented
Title: Ability to disable SSL verification for dev instances » Option to disable SSL verification for dev instances
afi13’s picture

afi13 commented
Status: Active » Needs review
afi13’s picture

afi13 commented
Assigned: afi13 » Unassigned
ad0z’s picture

ad0z commented
StatusFileSize
new search_api_opensearch-ssl-verification-option-3381020-6.patch5.84 KB

I've rerolled the @afi13 code to work on 2.x branch, additionally updated StandardConnector getConfiguration function because default value for websites with installed module already was not picked up. I believe it's expecting way to resolve this issue.

kim.pepper made their first commit to this issue’s fork.

kim.pepper changed the visibility of the branch 3.x to hidden.

kim.pepper opened merge request !69

kim.pepper changed the visibility of the branch 3381020-ability-to-disable to hidden.

kim.pepper’s picture

kim.pepper
Primary language English
Location 🏄‍♂️🇦🇺Sydney, Australia
commented

Created a MR for the 3.x branch.

  • kim.pepper committed 74e55358 on 3.x 
    Issue #3381020 by afi13, kim.pepper: Option to disable SSL verification...

  • kim.pepper committed 1d58c66a on 2.x 
    Issue #3381020 by afi13, kim.pepper: Option to disable SSL verification...
kim.pepper’s picture

kim.pepper
Primary language English
Location 🏄‍♂️🇦🇺Sydney, Australia
commented
Status: Needs review » Fixed

Committed to 2.x and 3.x Thanks!

kim.pepper opened merge request !70

kim.pepper closed merge request !70

kim.pepper’s picture

kim.pepper
Primary language English
Location 🏄‍♂️🇦🇺Sydney, Australia
commented
Version: 2.0.0-beta2 » 3.x-dev

  • kim.pepper committed 3238d4cb on 2.x 
    Revert "Issue #3381020 by afi13, kim.pepper: Option to disable SSL...
kim.pepper’s picture

kim.pepper
Primary language English
Location 🏄‍♂️🇦🇺Sydney, Australia
commented

Reverting this as we don't have an update hook to set the default value.

kim.pepper’s picture

kim.pepper
Primary language English
Location 🏄‍♂️🇦🇺Sydney, Australia
commented
Status: Fixed » Needs work

ad0z changed the visibility of the branch 3381020-disable-ssl-verification-option to hidden.

ad0z opened merge request !73

ad0z changed the visibility of the branch 3381020-disable-ssl to hidden.

ad0z’s picture

ad0z commented
Status: Needs work » Needs review

@kim.pepper created merge request with hook update. https://git.drupalcode.org/project/search_api_opensearch/-/merge_request...

  • kim.pepper committed 08fad842 on 2.x authored by ad0z 
    Issue #3381020 by kim.pepper, afi13: Option to disable SSL verification...
kim.pepper’s picture

kim.pepper
Primary language English
Location 🏄‍♂️🇦🇺Sydney, Australia
commented
Status: Needs review » Fixed

Committed to 2.x. Thanks!

Status: Fixed » Closed (fixed)

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