Repository: logging-log4j2 Updated Branches: refs/heads/master 41889364b -> f79570168
Make SocketAppender.advertisement final. Project: http://git-wip-us.apache.org/repos/asf/logging-log4j2/repo Commit: http://git-wip-us.apache.org/repos/asf/logging-log4j2/commit/f7957016 Tree: http://git-wip-us.apache.org/repos/asf/logging-log4j2/tree/f7957016 Diff: http://git-wip-us.apache.org/repos/asf/logging-log4j2/diff/f7957016 Branch: refs/heads/master Commit: f7957016893bba6770143bb20b4b56ce70f76215 Parents: 4188936 Author: ggregory <[email protected]> Authored: Wed Jan 20 18:34:37 2016 -0800 Committer: ggregory <[email protected]> Committed: Wed Jan 20 18:34:37 2016 -0800 ---------------------------------------------------------------------- .../org/apache/logging/log4j/core/appender/SocketAppender.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/f7957016/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/SocketAppender.java ---------------------------------------------------------------------- diff --git a/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/SocketAppender.java b/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/SocketAppender.java index 9c21d4a..341536e 100644 --- a/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/SocketAppender.java +++ b/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/SocketAppender.java @@ -47,7 +47,7 @@ public class SocketAppender extends AbstractOutputStreamAppender<AbstractSocketM private static final long serialVersionUID = 1L; - private Object advertisement; + private final Object advertisement; private final Advertiser advertiser; protected SocketAppender(final String name, final Layout<? extends Serializable> layout, final Filter filter, @@ -60,6 +60,8 @@ public class SocketAppender extends AbstractOutputStreamAppender<AbstractSocketM configuration.put("contentType", layout.getContentType()); configuration.put("name", name); this.advertisement = advertiser.advertise(configuration); + } else { + this.advertisement = null; } this.advertiser = advertiser; }
