Closed (fixed)
Project:
Elasticsearch Connector
Version:
8.x-7.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
6 Oct 2021 at 11:39 UTC
Updated:
20 Jan 2023 at 11:24 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
nick_schuch commentedHere is a patch which sets the required curl options for basic authentication while leaving the "Digest" and "NTLM" options to be handled at the library level.
Comment #3
nick_schuch commentedUpdated patch with debug code removed (should have had that extra coffee).
Comment #4
nick_schuch commentedComment #5
nicksanta commentedRelated - we encountered a regression recently where changes to FILTER_VALIDATE_URL in PHP 7.4.21 caused our apps using
@symbols in the credentials to stop connecting to the cluster.https://bugs.php.net/bug.php?id=81122
Comment #6
sokru commentedGood catch! I was not able to apply the patch from #3, so I created a new one. I did minor coding style improvements and added `ext-curl` requirement for composer.json. Most of Linux distributions have php-curl installed by default, but since neither Drupal core or dependent PHP-libraries require ext-curl, it might save some troubleshooting time from developers trying this module without php-curl installed, if they face the problem already on
composer require drupal/elasticsearch_connectorphase.Comment #8
sokru commented