[
https://issues.apache.org/jira/browse/SOLR-14714?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17179251#comment-17179251
]
Erick Erickson commented on SOLR-14714:
---------------------------------------
I looked over these changes and it looks good to me. I'll check it in in a
couple of days absent objections. I don't have a Windows machine to test with,
but Endika is quite correct that the variable is used before it's set.
> Solr.cmd in windows loads the incorrect jetty module when using java>=9
> -----------------------------------------------------------------------
>
> Key: SOLR-14714
> URL: https://issues.apache.org/jira/browse/SOLR-14714
> Project: Solr
> Issue Type: Bug
> Security Level: Public(Default Security Level. Issues are Public)
> Components: scripts and tools
> Affects Versions: 8.6
> Environment: Windows environment running in Solr Cloud mode with SSL.
> Reporter: Endika Posadas
> Priority: Major
> Attachments: load_java_info_first.patch
>
>
> In Solr.cmd, when using SSL, there is a check to verify what version of java
> solr is running with. If this version of Java is greater or equal java 9 it
> will load the jetty https module while for java 8 it will use https8.
> However, this java version check is done before the java major version
> variable has been assigned. As a result, Solr in windows doesn't work when
> SSL is enabled.
>
> To fix this issue, it is enough if java checks are done before SSL checks.
>
> I have attached a patch with the modifications.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org
