janhoy commented on PR #2540:
URL: https://github.com/apache/solr/pull/2540#issuecomment-2232999853
> looks like logic to identify a nodename assumes one `_` and we have two
`_` chars!
Yea, and it looks like the context path gets doubled when invoking the
create tool (see the `/box/box` path):
```
2> ERROR: Error from server at
http://127.0.0.1:50336/box/box/admin/info/system?wt=javabin&version=2: Expected
mime type in [application/vnd.apache.solr.javabin, application/octet-stream]
but got text/html. <html>
```
Smells like a followup bug related to the normalization of baseUrl, where I
added logic to be able to strip not only `/solr` but any context path when
computing the true base url. But something is still fishy.
Hard to reproduce in IntelliJ I believe, since this test fails due to the
"lock factory vs RAMDirectory" issue. You could temporarily patch the tests's
`solrconfig.xml` to using `MockDirectoryFactory` (which is set by gradle but
not by IntelliJ), to have the test pass and be able to step through it.
Probably need to pass some sys props to the test in IntellJ to reproduce the
exact same condition..
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
