Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "added logback xml config file" #957

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

Artur-
Copy link

@Artur- Artur- commented Sep 10, 2021

Libraries should not contain logging configuration. They interfere with application logging configurations.

This reverts commit 6a0629a.

Fixes #956

Libraries should not contain logging configuration. They interfere with application logging configurations.

This reverts commit 6a0629a.

Fixes swagger-api#956
@Artur-
Copy link
Author

Artur- commented Oct 6, 2021

How about this @HugoMario? If you try to use any Swagger version newer than 1.0.20 it eats all Spring Boot log output

@Artur-
Copy link
Author

Artur- commented Aug 10, 2022

This has soon been broken now for a year. Can we remove the logback file? Does it break anything else if it is removed?

You can see the problem pretty clearly if you

  1. Open https://start.spring.io/
  2. Download
  3. Run mvn spring-boot:run and watch the output
  4. Stop Spring Boot and add the dependency
<dependency>
    <groupId>io.swagger.codegen.v3</groupId>
    <artifactId>swagger-codegen-generators</artifactId>
    <version>1.0.34</version>
</dependency>
  1. Run mvn spring-boot:run and watch the output

Without Swagger the output is

[INFO] --- spring-boot-maven-plugin:2.7.2:run (default-cli) @ demo ---
[INFO] Attaching agents: []

  .   ____          _            __ _ _
 /\\ / ___'_ __ _ _(_)_ __  __ _ \ \ \ \
( ( )\___ | '_ | '_| | '_ \/ _` | \ \ \ \
 \\/  ___)| |_)| | | | | || (_| |  ) ) ) )
  '  |____| .__|_| |_|_| |_\__, | / / / /
 =========|_|==============|___/=/_/_/_/
 :: Spring Boot ::                (v2.7.2)

2022-08-10 09:05:58.355  INFO 96007 --- [           main] com.example.demo.DemoApplication         : Starting DemoApplication using Java 17.0.1 on hostname.com with PID 96007 (/directory/demo/target/classes started by user in /directory/demo)
2022-08-10 09:05:58.356  INFO 96007 --- [           main] com.example.demo.DemoApplication         : No active profile set, falling back to 1 default profile: "default"
2022-08-10 09:05:58.559  INFO 96007 --- [           main] com.example.demo.DemoApplication         : Started DemoApplication in 0.337 seconds (JVM running for 0.462)
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------

With Swagger it is

[INFO] --- spring-boot-maven-plugin:2.7.2:run (default-cli) @ demo ---
[INFO] Attaching agents: []

  .   ____          _            __ _ _
 /\\ / ___'_ __ _ _(_)_ __  __ _ \ \ \ \
( ( )\___ | '_ | '_| | '_ \/ _` | \ \ \ \
 \\/  ___)| |_)| | | | | || (_| |  ) ) ) )
  '  |____| .__|_| |_|_| |_\__, | / / / /
 =========|_|==============|___/=/_/_/_/
 :: Spring Boot ::                (v2.7.2)

[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Versions 1.0.21+ prevent any log output from Spring Boot applications
1 participant