Module doesn’t support authentication about silverstripe-solr-search

[Firesphere]

Yep, that should be fixed indeed. I've not personally dealt with authentication but any input is welcome!

[Firesphere]

What do you mean with the parts that don't work?
You mean, those two are not passing the credentials correctly?

[kinglozzer]

What do you mean with the parts that don't work?
You mean, those two are not passing the credentials correctly?

Yeah exactly, they manually build a Guzzle Client instance and ignore the rest of the configuration. Normally, everything is passed to Solarium: https://github.com/Firesphere/silverstripe-solr-search/blob/87dbf70/src/Services/SolrCoreService.php#L95-L96 so it’s easy to include username/password parameters

[Firesphere]

They're moving to PSR18 connector, so it may need a bit more work. If you have any idea or plans, please update on tag 1.0, so we can merge upstream

[Firesphere]

Documentation on Auth is quite "meh" on Solarium, it seems to need more than just a little bit.

I'd appreciate any help! I've not yet properly figured it out.

[Firesphere]

@kinglozzer Could you provide us with some (pseudo) code on what your hack is?

I've gotten a bit a better understanding on how to do the authentication now, but I'm still not sure how to use it with Solarium itself. It seems to be in the adapter instead of the connection, which is a tad confusing

[kinglozzer]

I’m afraid not, sorry! I directly modified the two methods linked in the original issue just to test it, but eventually I removed auth entirely and went for IP address-based restrictions instead.

SolrCoreService did work, so I think the Solarium part is fine (it just passes the $config array which contains anything/everything you want to add in YAML). The problem with the two other methods is that they manually construct Guzzle client instances, so the need authentication parameters (username/password, I think that’s all?) added manually. I can’t remember if they’re added as headers or GET/POST vars...

[Firesphere]

I'm trying to get this solved, but my main problem at the moment is getting Solr to cooperate! :D

[Firesphere]

@kinglozzer the two methods you mentioned aren't available in the Solarium module, so that's why it's "hacky"

I'm "hacking" my way around it now, and then see if I can get it to work. BasicAuth only for now

[Firesphere]

@kinglozzer Could you try the branch dev-hans/authentication and see if it solves the issues you had?

[kinglozzer]

Thanks @Firesphere, workloads are high at the moment so I might not get a chance to test for a little while. I’ve added a couple of comments to the commits but the approach looks good

[Firesphere]

Please continue discussion in the Pull Request