Eric Pugh
This has been fixed! Please reopen if you disagree!
I took a quick look and noticed that I did NOT write any unit tests for this code... bad @epugh ! So, going to back fill in some tests...
Will check in a few hours. Thanks for doing this administrivia to keep our site up to date!
Can you hook me up with an example? Like, are you thinking that instead of `reader:password` you want to support `re@der:p@assword` and have that work?
So, I *think* that with the release of 7.12.3 and the fixes for Solr and basic auth, that you don't need this because instead of the basic auth in the...
I've now pushed the changes to app.quepid.com as well...
Dug into this, and yeah, it's kind of a pain... So, while this doesn't SOLVE the problem, it at least warns the user and prvents you from using passwords that...
@dsmiley before I move forward on this (and maybe @janhoy you have an opinoin), the PR as is keeps the custom properties being loaded in for a core... I want...
Since `solrcore.properties` is deprecated, but core "properties" file support hasn't, I'm going to move forward with just this removal to get in under the solr 10 deadline.
I am a bit ambivalent... I believe Claude, however I really dislike `We avoid using cluster.getSolrClient() because it may have a background refresh job` because how would any human being...