[ECOMMERCE-537] magnolia-ecommerce-commercetools-connector contains TRACE level log42.xml Created: 13/Feb/23  Updated: 24/Apr/23  Resolved: 24/Apr/23

Status: Closed
Project: E-commerce
Component/s: connector
Affects Version/s: 1.3.3
Fix Version/s: None

Type: Bug Priority: Neutral
Reporter: Joerg von Frantzius Assignee: Unassigned
Resolution: Duplicate Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
duplicate
duplicates ECOMMERCE-520 Remove "info.magnolia" from log4j2.xm... Closed
Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Bug DoR:
[ ]* Steps to reproduce, expected, and actual results filled
[ ]* Affected version filled
Date of First Response:
Epic Link: AuthorX Support
Team: AuthorX

 Description   

Steps to reproduce

  1. Add following dependency to your WAR
     
    1. <dependency>            
         <groupId>info.magnolia.ecommerce</groupId>
         <artifactId>magnolia-ecommerce-commercetools-connector</artifactId>            
         <version>1.3.3</version>
      </dependency> 
  1. Start with `-Dlog4j2.debug=true` and observe
    1. [ESC[1;34mINFOESC[m] [talledLocalContainer] DEBUG StatusLogger Reconfiguration complete for context[name=c4c0b41] at URI jar:file:/Users/joerg.frantzius/git/magkit/showcase/target/catalina-author/webapps/author/WEB-INF/lib/magnolia-eco
      mmerce-commercetools-connector-1.3.3.jar!/log4j2.xml (org.apache.logging.log4j.core.LoggerContext@400964c9) with optional ClassLoader: null
  1. See unwanted TRACE level log for your webapp

.. Logs, screenshots, gifs...

Expected results

Default level log out in stdout, or as configured in /WEB-INF/config/default/log4j2.xml

Actual results

Tons of TRACE level log output in stdout

This is because https://git.magnolia-cms.com/projects/ADDON/repos/ecommerce/browse/magnolia-ecommerce-commercetools-connector/src/main/resources/log4j2.xml?at=refs%2Ftags%2Fmagnolia-ecommerce-1.3.3 exists and contains 

    <Root level="ALL">
      <AppenderRef ref="console"/>
    </Root> 


 Comments   
Comment by Andrei Ichimescu [ 24/Apr/23 ]

We have removed the log4j config from e-commerce 1.3.4: https://git.magnolia-cms.com/projects/ADDON/repos/ecommerce/browse/magnolia-ecommerce-commercetools-connector/src/main/resources/log4j2.xml?at=refs%2Ftags%2Fmagnolia-ecommerce-1.3.4
as part of ECOMMERCE-520

Generated at Mon Feb 12 01:45:02 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.