Feel free as RM to release without these changes and fold into 8.1. But here is a patch with re-phrasing of the two paragraphs I mentioned, as well as backport SOLR-12809, in case of a potential re-spin. https://gist.github.com/ed99d0945de112e05e1d1ff77772ce6a
-- Jan Høydahl, search solution architect Cominvent AS - www.cominvent.com > 3. jun. 2019 kl. 14:54 skrev Cassandra Targett <casstarg...@gmail.com>: > > Jan, would you please suggest exact wording to correct the error you pointed > out (quoted below). Since I clearly don’t understand the change, and this is > months delayed already, that is likely be the fastest option to get a respin > started this week. Otherwise, it will need to wait for me to wade back into > the topic to see if I can understand what the correct wording should be. > > Thank you, > Cassandra > On May 31, 2019, 3:37 PM -0500, Jan Høydahl <jan....@cominvent.com>, wrote: >> Viewed on a smartphone :) >> >> Comments: >> >> Major changes section: >> >>> The Basic Authentication plugin now has an option forwardCredentials to >>> prevent Basic Auth headers from being sent for inter-node requests. >> >> >> This is wrong. Basic Auth never sent basicauth headers inter-node, and >> adding the “forwardcredentials” option lets user-originated requests be >> forwarded with original basicauth credentials instead of PKI. >>> >
