External-Mirrors/keycloak
2
0
Fork 0
mirror of https://github.com/keycloak/keycloak.git synced 2026-01-09 23:12:06 -03:30
Code Issues Packages Projects Releases Wiki Activity

Unable to set '--log-syslog-max-length' property (#36252)

Browse Source 
Fixes #35386

Signed-off-by: Martin Bartoš <mabartos@redhat.com>
This commit is contained in:
Martin Bartoš 2025-01-08 15:41:46 +01:00 committed by GitHub
parent 3db9689010
commit d7d2a76676
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
3 changed files with 28 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
quarkus/config-api/src/main/java/org/keycloak/config/LoggingOptions.java
View File

@ -182,7 +182,7 @@ public class LoggingOptions {
.defaultValue(SyslogHandler.SyslogType.RFC5424.toString().toLowerCase())
.build();
public static final Option<MemorySize> LOG_SYSLOG_MAX_LENGTH = new OptionBuilder<>("log-syslog-max-length", MemorySize.class)
public static final Option<String> LOG_SYSLOG_MAX_LENGTH = new OptionBuilder<>("log-syslog-max-length", String.class)
.category(OptionCategory.LOGGING)
// based on the 'quarkus.log.syslog.max-length' property
.description("Set the maximum length, in bytes, of the message allowed to be sent. The length includes the header and the message. " +

11
quarkus/runtime/src/main/java/org/keycloak/quarkus/runtime/configuration/mappers/LoggingPropertyMappers.java
View File

@ -12,6 +12,7 @@ import java.util.function.BiFunction;
import java.util.logging.Level;
import java.util.stream.Stream;
import io.quarkus.runtime.configuration.MemorySizeConverter;
import org.jboss.logmanager.LogContext;
import org.keycloak.config.LoggingOptions;
import org.keycloak.config.Option;
@ -144,6 +145,7 @@ public final class LoggingPropertyMappers {
fromOption(LoggingOptions.LOG_SYSLOG_MAX_LENGTH)
.isEnabled(LoggingPropertyMappers::isSyslogEnabled, SYSLOG_ENABLED_MSG)
.to("quarkus.log.syslog.max-length")
.validator(LoggingPropertyMappers::validateSyslogMaxLength)
.paramLabel("max-length")
.build(),
fromOption(LoggingOptions.LOG_SYSLOG_PROTOCOL)
@ -295,4 +297,13 @@ public final class LoggingPropertyMappers {
private static String upperCase(String value, ConfigSourceInterceptorContext context) {
return value.toUpperCase(Locale.ROOT);
}
private static void validateSyslogMaxLength(String value) {
var converter = new MemorySizeConverter();
try {
converter.convert(value);
} catch (IllegalArgumentException e) {
throw new PropertyException(String.format("Invalid value for option '--log-syslog-max-length': %s", e.getMessage()));
}
}
}

16
quarkus/runtime/src/test/java/org/keycloak/quarkus/runtime/cli/PicocliTest.java
View File

@ -20,6 +20,7 @@ package org.keycloak.quarkus.runtime.cli;
import static org.hamcrest.MatcherAssert.assertThat;
import static org.hamcrest.Matchers.containsString;
import static org.hamcrest.Matchers.not;
import static org.hamcrest.Matchers.nullValue;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertFalse;
import static org.junit.Assert.assertNull;
@ -445,4 +446,19 @@ public class PicocliTest extends AbstractConfigurationTest {
containsString("Usage of the default value for the db option in the production profile is deprecated. Please explicitly set the db instead."));
}
@Test
public void syslogMaxLengthMemorySize() {
NonRunningPicocli nonRunningPicocli = pseudoLaunch("start-dev", "--log=syslog", "--log-syslog-max-length=60k");
assertEquals(CommandLine.ExitCode.OK, nonRunningPicocli.exitCode);
assertEquals("60k", nonRunningPicocli.config.getConfigValue("quarkus.log.syslog.max-length").getValue());
nonRunningPicocli = pseudoLaunch("start-dev", "--log=syslog");
assertEquals(CommandLine.ExitCode.OK, nonRunningPicocli.exitCode);
assertThat(nonRunningPicocli.config.getConfigValue("quarkus.log.syslog.max-length").getValue(), nullValue());
nonRunningPicocli = pseudoLaunch("start-dev", "--log=syslog", "--log-syslog-max-length=wrong");
assertEquals(CommandLine.ExitCode.USAGE, nonRunningPicocli.exitCode);
assertThat(nonRunningPicocli.getErrString(), containsString(
"Invalid value for option '--log-syslog-max-length': value wrong not in correct format (regular expression): [0-9]+[BbKkMmGgTtPpEeZzYy]?"));
}
}