[PR] Improved: Remove OFBizDynamicThresholdFilter (OFBIZ-13726) [ofbiz-framework]

["mridulpathak (via GitHub)" <gi...@apache.org>]

mridulpathak opened a new pull request, #2426:
URL: https://github.com/apache/ofbiz-framework/pull/2426

   OFBizDynamicThresholdFilter was declared for the whole Log4j configuration, so it ran on every logging call, building a new context data map each time, only to keep the getJs requests at the ERROR level; it also relied on internal Log4j classes and on the plugin package scanning in log4j2.xml. This removes it, the ThreadContext "uri" entries set by ControlFilter and CatalogUrlFilter to feed it, and makes log4j-core a runtime dependency (and a test one for EntityEcaUtilConcurrencyTest). The getJs requests now log like any other request.
   
   Thanks: Jacopo Cappellato for the suggestion
   


-- 
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: notifications-unsubscribe@ofbiz.apache.org

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

Re: [PR] Improved: Remove OFBizDynamicThresholdFilter (OFBIZ-13726) [ofbiz-framework]

["mridulpathak (via GitHub)" <gi...@apache.org>]

mridulpathak merged PR #2426:
URL: https://github.com/apache/ofbiz-framework/pull/2426


-- 
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: notifications-unsubscribe@ofbiz.apache.org

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