Skip to content

Fix logging bufio.Scanner bug.#2

Open
baux1980 wants to merge 1 commit intogreenbaum:masterfrom
baux1980:timeout-log-buffio
Open

Fix logging bufio.Scanner bug.#2
baux1980 wants to merge 1 commit intogreenbaum:masterfrom
baux1980:timeout-log-buffio

Conversation

@baux1980
Copy link

@baux1980 baux1980 commented Jun 12, 2023

A bug in used log lib github.com/sirupsen/logrus v1.9.0:

Error while reading from Writer: bufio.Scanner: token too long #1370

sirupsen/logrus#1370

That was closing unexpectly the stderr log socket provided by containerpilot to managed services.

Good catch by Ema Musella which founds the exact error message that lead us to problem solving.

The bug is fixed by updating logrus to release 1.9.3.

A bug in used log lib github.com/sirupsen/logrus v1.9.0:

   Error while reading from Writer: bufio.Scanner: token too long #1370

 sirupsen/logrus#1370

That was closing unexpectly the stderr log socket provided by containerpilot
to managed services.

Good catch by Ema Musella which founds the exact error message that lead us
to problem solving.

The bug is fixed by updating logrus to release 1.9.3.
@baux1980
Copy link
Author

any news or comment about this fix?

Best regards.

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.

1 participant