You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@solr.apache.org by GitBox <gi...@apache.org> on 2021/12/05 05:22:28 UTC

[GitHub] [solr] gus-asf opened a new pull request #445: SOLR-15590 Fix SSL, init order misplaced by prior commit for this ticket.

gus-asf opened a new pull request #445:
URL: https://github.com/apache/solr/pull/445


   Moves init into jetty-ssl.xml to ensure ssl is ready before any of our https request related code needs it, and remove such concerns form request and servlet related classes.
   


-- 
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: issues-unsubscribe@solr.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@solr.apache.org
For additional commands, e-mail: issues-help@solr.apache.org


[GitHub] [solr] gus-asf commented on pull request #445: SOLR-15590 Fix SSL, init order misplaced by prior commit for this ticket.

Posted by GitBox <gi...@apache.org>.
gus-asf commented on pull request #445:
URL: https://github.com/apache/solr/pull/445#issuecomment-986304968


   To test this locally I ran through the (9.0) ref guide setup for SSL with self signed cert. I found that this allowed https generally. I also found a bug that is unrelated to my changes ( exists even if I revert to version prior to my merging 15590 ) in which the baseUrl calcuation gets the wrong scheme. Was late when I finished this, but I'll file that now if I can't find it already reported.


-- 
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: issues-unsubscribe@solr.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@solr.apache.org
For additional commands, e-mail: issues-help@solr.apache.org


[GitHub] [solr] thelabdude commented on pull request #445: SOLR-15590 Fix SSL, init order misplaced by prior commit for this ticket.

Posted by GitBox <gi...@apache.org>.
thelabdude commented on pull request #445:
URL: https://github.com/apache/solr/pull/445#issuecomment-986287527


   I have a process to test it (which is how I found it previously), will try later today


-- 
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: issues-unsubscribe@solr.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@solr.apache.org
For additional commands, e-mail: issues-help@solr.apache.org


[GitHub] [solr] gus-asf commented on pull request #445: SOLR-15590 Fix SSL, init order misplaced by prior commit for this ticket.

Posted by GitBox <gi...@apache.org>.
gus-asf commented on pull request #445:
URL: https://github.com/apache/solr/pull/445#issuecomment-986307923


   https://issues.apache.org/jira/browse/SOLR-15835 filed


-- 
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: issues-unsubscribe@solr.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@solr.apache.org
For additional commands, e-mail: issues-help@solr.apache.org


[GitHub] [solr] gus-asf merged pull request #445: SOLR-15590 Fix SSL, init order misplaced by prior commit for this ticket.

Posted by GitBox <gi...@apache.org>.
gus-asf merged pull request #445:
URL: https://github.com/apache/solr/pull/445


   


-- 
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: issues-unsubscribe@solr.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@solr.apache.org
For additional commands, e-mail: issues-help@solr.apache.org