From da23031b4e448ea5e262023114da5e499700956e Mon Sep 17 00:00:00 2001
From: Burak KALAYCI <kalayciburak1996@gmail.com>
Date: Tue, 4 Aug 2026 14:20:26 +0300
Subject: [PATCH] Fix Log4j1SyslogLayout builder javadoc attributes

Remove copy-pasted includeNewLine/escapeNL docs from SyslogLayout and
document the attributes this builder actually exposes.

Fixes #4237

Signed-off-by: Burak KALAYCI <kalayciburak1996@gmail.com>
---
 .../org/apache/log4j/layout/Log4j1SyslogLayout.java   | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/log4j-1.2-api/src/main/java/org/apache/log4j/layout/Log4j1SyslogLayout.java b/log4j-1.2-api/src/main/java/org/apache/log4j/layout/Log4j1SyslogLayout.java
index 4a3dcc11625..65454185299 100644
--- a/log4j-1.2-api/src/main/java/org/apache/log4j/layout/Log4j1SyslogLayout.java
+++ b/log4j-1.2-api/src/main/java/org/apache/log4j/layout/Log4j1SyslogLayout.java
@@ -49,14 +49,19 @@
 public final class Log4j1SyslogLayout extends AbstractStringLayout {
 
     /**
-     * Builds a SyslogLayout.
+     * Builds a {@link Log4j1SyslogLayout}.
      * <p>The main arguments are</p>
      * <ul>
      * <li>facility: The Facility is used to try to classify the message.</li>
-     * <li>includeNewLine: If true a newline will be appended to the result.</li>
-     * <li>escapeNL: Pattern to use for replacing newlines.</li>
+     * <li>facilityPrinting: If true the facility name is included in the message.</li>
+     * <li>header: If true a header with local host name and timestamp is included.</li>
+     * <li>messageLayout: Optional layout used to render the message body.</li>
      * <li>charset: The character set.</li>
      * </ul>
+     * <p>
+     * Unlike {@link org.apache.logging.log4j.core.layout.SyslogLayout}, this layout does not support
+     * {@code includeNewLine} or {@code escapeNL} options.
+     * </p>
      * @param <B> the builder type
      */
     public static class Builder<B extends Builder<B>> extends AbstractStringLayout.Builder<B>