You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by jl...@apache.org on 2022/06/17 15:34:50 UTC

[ofbiz-plugins] branch trunk updated: Improved: Sorl external logging is not working, but a bit in console (OFBIZ-12646)

This is an automated email from the ASF dual-hosted git repository.

jleroux pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/ofbiz-plugins.git


The following commit(s) were added to refs/heads/trunk by this push:
     new 67cbf35e8 Improved: Sorl external logging is not working, but a bit in console (OFBIZ-12646)
67cbf35e8 is described below

commit 67cbf35e83068fcecc86af6a66ea2043e71ff544
Author: Jacques Le Roux <ja...@les7arts.com>
AuthorDate: Fri Jun 17 17:34:55 2022 +0200

    Improved: Sorl external logging is not working, but a bit in console (OFBIZ-12646)
    
    After removing useless system properties used to handle logging in
    OFBizSolrContextFilter.java with commit 102eb511531d64c131ed78fe536fc2857b7eb4b1
    that had a wrong commit title
    <<Improved: Unable to upload a file through ecommerce (OFBIZ 12636)>>
    and message that should have been:
    <<Remove useless system properties used to handle logging. Maybe previously this
    was working but it was then long ago.
    
    Also speeds things a bit while loading Solr by using the not so obvious
    SOLRHOME_ATTRIBUTE servlet attribute rather than the "solr/home" system property
    
    Note: this is a 1st cleaning step and we are still missing external logging>>
    
    I forgot to remove the related properties in solrconfig.properties, this does it
---
 solr/config/solrconfig.properties | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/solr/config/solrconfig.properties b/solr/config/solrconfig.properties
index b87a3052f..9df36d031 100644
--- a/solr/config/solrconfig.properties
+++ b/solr/config/solrconfig.properties
@@ -46,10 +46,5 @@ solr.client.connection.timeout=
 # If true, trust self signed certification, default is false.
 solr.client.trust.selfsigned.cert=true
 
-# Defines Solr specific log directory
-solr.log.dir=runtime/logs/solr
-# Defines Solr log level
-solr.log.level=INFO
-
 # Define the location of its home, speeds loading
 solr/home=/plugins/solr/home