cnoe-io / idpbuilder

Spin up a complete internal developer platform with only Docker required as a dependency.
https://cloud-native.slack.com/archives/C05TN9WFN5S
Apache License 2.0
200 stars 59 forks source link

[Suggestion]: Improve the format of the messages logged #395

Open cmoulliard opened 1 month ago

cmoulliard commented 1 month ago

Have you searched for this feature request?

Problem Statement

Suggestion

Improve the format of the messages logged which are really long to make them shorter as they :

Screenshot 2024-10-03 at 18 10 27

Note: In java, we log the messages as such and that improves the readability

__  ____  __  _____   ___  __ ____  ______ 
 --/ __ \/ / / / _ | / _ \/ //_/ / / / __/ 
 -/ /_/ / /_/ / __ |/ , _/ ,< / /_/ /\ \   
--\___\_\____/_/ |_/_/|_/_/|_|\____/___/   
2024-10-04 10:55:19,450 INFO  [io.quarkus] (main) code-with-quarkus 1.0.0-SNAPSHOT on JVM (powered by Quarkus 999-SNAPSHOT) started in 0.477s. Listening on: http://0.0.0.0:8080
2024-10-04 10:55:19,454 INFO  [io.quarkus] (main) Profile prod activated. 
2024-10-04 10:55:19,454 INFO  [io.quarkus] (main) Installed features: [argocd, cdi, kubernetes, rest, smallrye-context-propagation, vertx]
...

Possible Solution

n/a

Alternatives Considered

No response

nabuskey commented 1 month ago

Agreed. I'd like to improve the readability more. Implementation wise, it shouldn't be too difficult because we already have custom logger implemented. Just need format changes.

cmoulliard commented 1 month ago

I also notice that some messages are not logged as (I think so) still use Logger.V (see tls.go)