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

Suggestion: add container log support, as well as ANSI color removal #17

Open
WesternGun opened this issue Jan 9, 2023 · 1 comment

Comments

@WesternGun
Copy link

WesternGun commented Jan 9, 2023

Oftentimes currently, the logs come from docker containers or docker compose, with ANSI color chars(cyan in my example):

�[36mapp_1                        |�[0m 2023-01-09 19:59:02,911 DEBUG [org.apa.cam.pro.FilterProcessor]-{Camel (camel-1) thread #20 - Split}-Filter matches: true for exchange: Exchange[1B8CEEBBF0626A1-0000000000000007]

In that case, log4j patterns will not match, because it cannot understand ANSI color control chars, neither the app_1 | before each line. They mark the service name in docker-compose.yaml file.

It would be good to parse them, so that I don't have to find/replace them each time.

@grobmeier
Copy link
Member

Hi, @WesternGun, this sounds like a very good idea.
I am trying to get familiar with Chainsaw, and you could help me reproduce the issue. Can you create a demo container or Java classes that make exactly that log output?

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

No branches or pull requests

2 participants