From 3f17e42b38fba573cd3569e4213911d59ffe7c5c Mon Sep 17 00:00:00 2001 From: Yanming Zhou Date: Fri, 29 Nov 2024 11:25:47 +0800 Subject: [PATCH 1/2] Polish formatting of log configuration file 1. Remove space before `/>` to keep consistency. 2. Capitalize `filters` for Log4j2. 3. Uppercase log level to keep consistency. See gh-43322 --- .../boot/logging/log4j2/log4j2-file.xml | 24 +++++++++---------- .../boot/logging/log4j2/log4j2.xml | 24 +++++++++---------- .../boot/logging/logback/base.xml | 10 ++++---- .../boot/logging/logback/defaults.xml | 10 ++++---- .../src/test/resources/logback-broken.xml | 2 +- .../resources/logback-include-defaults.xml | 2 +- .../src/test/resources/logback-janino.xml | 2 +- .../src/test/resources/logback-nondefault.xml | 2 +- .../logback-springprofile-in-root.xml | 2 +- .../src/dockerTest/resources/logback.xml | 2 +- .../src/main/resources/log4j2.xml | 6 ++--- .../src/main/resources/logback.xml | 4 ++-- .../src/main/resources/logback-spring.xml | 6 ++--- 13 files changed, 48 insertions(+), 48 deletions(-) diff --git a/spring-boot-project/spring-boot/src/main/resources/org/springframework/boot/logging/log4j2/log4j2-file.xml b/spring-boot-project/spring-boot/src/main/resources/org/springframework/boot/logging/log4j2/log4j2-file.xml index 916f1126db61..e5e513dee2ea 100644 --- a/spring-boot-project/spring-boot/src/main/resources/org/springframework/boot/logging/log4j2/log4j2-file.xml +++ b/spring-boot-project/spring-boot/src/main/resources/org/springframework/boot/logging/log4j2/log4j2-file.xml @@ -20,22 +20,22 @@ - + - - - - - - - - - - - + + + + + + + + + + + diff --git a/spring-boot-project/spring-boot/src/main/resources/org/springframework/boot/logging/log4j2/log4j2.xml b/spring-boot-project/spring-boot/src/main/resources/org/springframework/boot/logging/log4j2/log4j2.xml index 239f2e35a34c..ee1044c36be1 100644 --- a/spring-boot-project/spring-boot/src/main/resources/org/springframework/boot/logging/log4j2/log4j2.xml +++ b/spring-boot-project/spring-boot/src/main/resources/org/springframework/boot/logging/log4j2/log4j2.xml @@ -10,22 +10,22 @@ - + - + - - - - - - - - - - + + + + + + + + + + diff --git a/spring-boot-project/spring-boot/src/main/resources/org/springframework/boot/logging/logback/base.xml b/spring-boot-project/spring-boot/src/main/resources/org/springframework/boot/logging/logback/base.xml index b1b17e289d92..04458c92a97e 100644 --- a/spring-boot-project/spring-boot/src/main/resources/org/springframework/boot/logging/logback/base.xml +++ b/spring-boot-project/spring-boot/src/main/resources/org/springframework/boot/logging/logback/base.xml @@ -5,12 +5,12 @@ Base logback configuration provided for compatibility with Spring Boot 1.1 --> - + - - + + - - + + diff --git a/spring-boot-project/spring-boot/src/main/resources/org/springframework/boot/logging/logback/defaults.xml b/spring-boot-project/spring-boot/src/main/resources/org/springframework/boot/logging/logback/defaults.xml index fa4006d2fc44..bad74fe1643d 100644 --- a/spring-boot-project/spring-boot/src/main/resources/org/springframework/boot/logging/logback/defaults.xml +++ b/spring-boot-project/spring-boot/src/main/resources/org/springframework/boot/logging/logback/defaults.xml @@ -5,11 +5,11 @@ Default logback configuration provided for import --> - - - - - + + + + + diff --git a/spring-boot-project/spring-boot/src/test/resources/logback-broken.xml b/spring-boot-project/spring-boot/src/test/resources/logback-broken.xml index 6411a145d452..7f8454941cff 100644 --- a/spring-boot-project/spring-boot/src/test/resources/logback-broken.xml +++ b/spring-boot-project/spring-boot/src/test/resources/logback-broken.xml @@ -6,6 +6,6 @@ - + diff --git a/spring-boot-project/spring-boot/src/test/resources/logback-include-defaults.xml b/spring-boot-project/spring-boot/src/test/resources/logback-include-defaults.xml index 76b3943cd2fc..0124d8a7e357 100644 --- a/spring-boot-project/spring-boot/src/test/resources/logback-include-defaults.xml +++ b/spring-boot-project/spring-boot/src/test/resources/logback-include-defaults.xml @@ -7,6 +7,6 @@ - + diff --git a/spring-boot-project/spring-boot/src/test/resources/logback-janino.xml b/spring-boot-project/spring-boot/src/test/resources/logback-janino.xml index 569b18e17ccb..5274e6f9693e 100644 --- a/spring-boot-project/spring-boot/src/test/resources/logback-janino.xml +++ b/spring-boot-project/spring-boot/src/test/resources/logback-janino.xml @@ -13,6 +13,6 @@ - + diff --git a/spring-boot-project/spring-boot/src/test/resources/logback-nondefault.xml b/spring-boot-project/spring-boot/src/test/resources/logback-nondefault.xml index da5f24ea1b11..5d1622c51841 100644 --- a/spring-boot-project/spring-boot/src/test/resources/logback-nondefault.xml +++ b/spring-boot-project/spring-boot/src/test/resources/logback-nondefault.xml @@ -6,6 +6,6 @@ - + diff --git a/spring-boot-project/spring-boot/src/test/resources/logback-springprofile-in-root.xml b/spring-boot-project/spring-boot/src/test/resources/logback-springprofile-in-root.xml index ea07eb8bd007..284c547a0b32 100644 --- a/spring-boot-project/spring-boot/src/test/resources/logback-springprofile-in-root.xml +++ b/spring-boot-project/spring-boot/src/test/resources/logback-springprofile-in-root.xml @@ -7,7 +7,7 @@ - + diff --git a/spring-boot-tests/spring-boot-integration-tests/spring-boot-launch-script-tests/src/dockerTest/resources/logback.xml b/spring-boot-tests/spring-boot-integration-tests/spring-boot-launch-script-tests/src/dockerTest/resources/logback.xml index 7c402db9a3fa..13e689a29304 100644 --- a/spring-boot-tests/spring-boot-integration-tests/spring-boot-launch-script-tests/src/dockerTest/resources/logback.xml +++ b/spring-boot-tests/spring-boot-integration-tests/spring-boot-launch-script-tests/src/dockerTest/resources/logback.xml @@ -5,6 +5,6 @@ - + diff --git a/spring-boot-tests/spring-boot-smoke-tests/spring-boot-smoke-test-actuator-log4j2/src/main/resources/log4j2.xml b/spring-boot-tests/spring-boot-smoke-tests/spring-boot-smoke-test-actuator-log4j2/src/main/resources/log4j2.xml index 1c84d286b093..238492a791c7 100644 --- a/spring-boot-tests/spring-boot-smoke-tests/spring-boot-smoke-test-actuator-log4j2/src/main/resources/log4j2.xml +++ b/spring-boot-tests/spring-boot-smoke-tests/spring-boot-smoke-test-actuator-log4j2/src/main/resources/log4j2.xml @@ -10,9 +10,9 @@ - - - + + + diff --git a/spring-boot-tests/spring-boot-smoke-tests/spring-boot-smoke-test-actuator/src/main/resources/logback.xml b/spring-boot-tests/spring-boot-smoke-tests/spring-boot-smoke-test-actuator/src/main/resources/logback.xml index 5c19b4eed630..613cb692a5e3 100644 --- a/spring-boot-tests/spring-boot-smoke-tests/spring-boot-smoke-test-actuator/src/main/resources/logback.xml +++ b/spring-boot-tests/spring-boot-smoke-tests/spring-boot-smoke-test-actuator/src/main/resources/logback.xml @@ -1,7 +1,7 @@ - + - + diff --git a/spring-boot-tests/spring-boot-smoke-tests/spring-boot-smoke-test-logback/src/main/resources/logback-spring.xml b/spring-boot-tests/spring-boot-smoke-tests/spring-boot-smoke-test-logback/src/main/resources/logback-spring.xml index 0a8ee5a84673..baf8bc039fc1 100644 --- a/spring-boot-tests/spring-boot-smoke-tests/spring-boot-smoke-test-logback/src/main/resources/logback-spring.xml +++ b/spring-boot-tests/spring-boot-smoke-tests/spring-boot-smoke-test-logback/src/main/resources/logback-spring.xml @@ -1,8 +1,8 @@ - - + + - + From cc274ccdcab4700c765b0bcb19a14be634a50598 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?St=C3=A9phane=20Nicoll?= Date: Wed, 4 Dec 2024 10:33:58 +0100 Subject: [PATCH 2/2] Polish "Polish formatting of log configuration file" See gh-43322 --- .../src/test/resources/logback-without-extension | 2 +- .../boot/logging/logback/include-with-profile.xml | 2 +- .../boot/logging/logback/include-with-property.xml | 2 +- .../boot/logging/logback/multi-profile-names.xml | 4 ++-- .../org/springframework/boot/logging/logback/nested.xml | 4 ++-- .../boot/logging/logback/production-profile.xml | 4 ++-- .../boot/logging/logback/profile-expression.xml | 4 ++-- .../boot/logging/logback/profile-in-include.xml | 4 ++-- .../boot/logging/logback/property-default-value.xml | 2 +- .../springframework/boot/logging/logback/property-in-if.xml | 6 +++--- .../boot/logging/logback/property-in-include.xml | 4 ++-- .../org/springframework/boot/logging/logback/property.xml | 2 +- 12 files changed, 20 insertions(+), 20 deletions(-) diff --git a/spring-boot-project/spring-boot/src/test/resources/logback-without-extension b/spring-boot-project/spring-boot/src/test/resources/logback-without-extension index 68080b7c975b..1d55bbaf83fb 100644 --- a/spring-boot-project/spring-boot/src/test/resources/logback-without-extension +++ b/spring-boot-project/spring-boot/src/test/resources/logback-without-extension @@ -6,6 +6,6 @@ - + diff --git a/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/include-with-profile.xml b/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/include-with-profile.xml index 12772f0a3b27..231de8f580ae 100644 --- a/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/include-with-profile.xml +++ b/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/include-with-profile.xml @@ -1,5 +1,5 @@ - + \ No newline at end of file diff --git a/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/include-with-property.xml b/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/include-with-property.xml index c7e095d1596d..74f08edcb49e 100644 --- a/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/include-with-property.xml +++ b/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/include-with-property.xml @@ -1,3 +1,3 @@ - + \ No newline at end of file diff --git a/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/multi-profile-names.xml b/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/multi-profile-names.xml index 480897e67423..4f785c163a02 100644 --- a/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/multi-profile-names.xml +++ b/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/multi-profile-names.xml @@ -1,7 +1,7 @@ - + - + diff --git a/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/nested.xml b/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/nested.xml index 9ad8b0456803..247229cc0055 100644 --- a/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/nested.xml +++ b/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/nested.xml @@ -1,9 +1,9 @@ - + - + diff --git a/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/production-profile.xml b/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/production-profile.xml index 8b263afd5ddf..bb3c8aae3cba 100644 --- a/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/production-profile.xml +++ b/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/production-profile.xml @@ -1,7 +1,7 @@ - + - + diff --git a/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/profile-expression.xml b/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/profile-expression.xml index 448bb017a5e4..c11d7f9f6599 100644 --- a/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/profile-expression.xml +++ b/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/profile-expression.xml @@ -1,7 +1,7 @@ - + - + diff --git a/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/profile-in-include.xml b/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/profile-in-include.xml index 3c4c9ec17f6f..a03c5d926883 100644 --- a/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/profile-in-include.xml +++ b/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/profile-in-include.xml @@ -1,5 +1,5 @@ - - + + diff --git a/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/property-default-value.xml b/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/property-default-value.xml index dd00a99b875d..5fcf44025c48 100644 --- a/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/property-default-value.xml +++ b/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/property-default-value.xml @@ -1,6 +1,6 @@ - + diff --git a/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/property-in-if.xml b/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/property-in-if.xml index b16fde51f3e6..92bb19cd9e7b 100644 --- a/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/property-in-if.xml +++ b/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/property-in-if.xml @@ -1,10 +1,10 @@ - - + + - + diff --git a/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/property-in-include.xml b/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/property-in-include.xml index ca2c2705b4e8..f367385c99e3 100644 --- a/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/property-in-include.xml +++ b/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/property-in-include.xml @@ -1,5 +1,5 @@ - - + + diff --git a/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/property.xml b/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/property.xml index 15ef50ae3293..a574d13df7b2 100644 --- a/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/property.xml +++ b/spring-boot-project/spring-boot/src/test/resources/org/springframework/boot/logging/logback/property.xml @@ -1,6 +1,6 @@ - +