-
Notifications
You must be signed in to change notification settings - Fork 29
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
NC 26 edition does not work with basi auth #256
Comments
Thanks a lot - its working now! |
Yeah this is just some ducktape ;) We'll have to wait for the maintainber to respond for a decent fix. |
+1 - hardcoded the credentials for now in my instance, and indexing works now. |
PR is here #261 |
ArtificialOwl
added a commit
that referenced
this issue
Aug 8, 2023
Authentication for ES8 added Closing #256
ArtificialOwl
added a commit
that referenced
this issue
Aug 8, 2023
[stable27] Authentication for ES8 added Closing #256
ArtificialOwl
added a commit
that referenced
this issue
Aug 8, 2023
[stable26] Authentication for ES8 added Closing #256
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Seems that http(s)://username:password@host:port does not work with the latest elesticsearch php connector. Line 384/385 of fulltextsearch_elasticsearch/lib/Platform/ElasticSearchPlatform.php should read:
$cb = ClientBuilder::create() ->setHosts($hosts) ->setBasicAuthentication($username, $password) ->setRetries(3);
Guess you could extract that from the hostname, but since you can provide several hosts, I guess it would be better and easier to add an optional username and password field to the config form? :)
The text was updated successfully, but these errors were encountered: