Looks like this would also fix https://issues.apache.org/jira/browse/LOG4J2-1573. I'll try that shortly.
On 09/12/2016 07:55 AM, Gary Gregory wrote:
I understand now, thank you. All build methods do not use the same default. I'll remove the @Required tomorrow. Gary On Sep 11, 2016 9:11 PM, "Matt Sicker" <[email protected] <mailto:[email protected]>> wrote: I mean if you do something like this: @PluginElement("Layout") @Required private Layout layout = PatternLayout.defaultLayout(); Then it should work. But if you defer the creation of a default layout until you execute the build() method, then the validator will err out before build() is called. On 11 September 2016 at 21:20, Gary Gregory <[email protected] <mailto:[email protected]>> wrote: HI Matt, Right now, all of the build() methods handle null layouts by using a default layout. That works for programmatic configs. But in order for that to work from a config file, the @Required needs to be removed. I feel like I am not understanding something in your message :-( Gary On Sun, Sep 11, 2016 at 11:43 AM, Matt Sicker <[email protected] <mailto:[email protected]>> wrote: I thought that @Required would check the field at build time, not injection time. If the field was set to null, then you're going to have a bad time. On 11 September 2016 at 10:50, Gary Gregory <[email protected] <mailto:[email protected]>> wrote: I forgot to say that: org.apache.logging.log4j.core.appender.ConsoleAppender.Builder.build() org.apache.logging.log4j.core.appender.FileAppender.Builder.build() org.apache.logging.log4j.core.appender.RollingFileAppender.Builder.build() org.apache.logging.log4j.core.appender.SocketAppender.Builder.build() All provide default layouts. Gary On Sun, Sep 11, 2016 at 8:48 AM, Gary Gregory <[email protected] <mailto:[email protected]>> wrote: I'm pretty sure we no longer need @Required on layout on org.apache.logging.log4j.core.appender.AbstractAppender.Builder.layout. Would it be OK to say that an appender should provide a default layout? Gary -- E-Mail: [email protected] <mailto:[email protected]> | [email protected] <mailto:[email protected]> Java Persistence with Hibernate, Second Edition <http://www.manning.com/bauer3/> JUnit in Action, Second Edition <http://www.manning.com/tahchiev/> Spring Batch in Action <http://www.manning.com/templier/> Blog: http://garygregory.wordpress.com <http://garygregory.wordpress.com/> Home: http://garygregory.com/ Tweet! http://twitter.com/GaryGregory -- E-Mail: [email protected] <mailto:[email protected]> | [email protected] <mailto:[email protected]> Java Persistence with Hibernate, Second Edition <http://www.manning.com/bauer3/> JUnit in Action, Second Edition <http://www.manning.com/tahchiev/> Spring Batch in Action <http://www.manning.com/templier/> Blog: http://garygregory.wordpress.com <http://garygregory.wordpress.com/> Home: http://garygregory.com/ Tweet! http://twitter.com/GaryGregory -- Matt Sicker <[email protected] <mailto:[email protected]>> -- E-Mail: [email protected] <mailto:[email protected]> | [email protected] <mailto:[email protected]> Java Persistence with Hibernate, Second Edition <http://www.manning.com/bauer3/> JUnit in Action, Second Edition <http://www.manning.com/tahchiev/> Spring Batch in Action <http://www.manning.com/templier/> Blog: http://garygregory.wordpress.com <http://garygregory.wordpress.com/> Home: http://garygregory.com/ Tweet! http://twitter.com/GaryGregory -- Matt Sicker <[email protected] <mailto:[email protected]>>
-- aixigo AG - financial solutions & technology Karl-Friedrich-Straße 68, 52072 Aachen, Germany fon: +49 (0)241 559709-65, fax: +49 (0)241 559709-99 eMail: [email protected], web: http://www.aixigo.de Amtsgericht Aachen - HRB 8057 Vorstand: Erich Borsch, Christian Friedrich, Tobias Haustein Vors. des Aufsichtsrates: Prof. Dr. Rüdiger von Nitzsch --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
