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

Application log lines > 8KB are split across multiple log lines by Passenger #2413

Open
jtomson opened this issue Feb 11, 2022 · 6 comments
Open

Comments

@jtomson
Copy link
Contributor

jtomson commented Feb 11, 2022

Question 1: What is the problem?

  • What is the expected behavior?
    A single logline from rails will not be split across multiple loglines

  • What is the actual behavior?
    A single logline from rails > 8KB will be split across multiple loglines

  • How can we reproduce it? Please try to provide a sample application (or Virtual Machine) demonstrating the issue. Otherwise, if we can't reproduce it, we might have to ask you a number of followup questions or run certain commands to try and figure out the problem.

Log a single statement > 8KB in a rails process hosted by passenger. Passenger will have split this into > 1 logline as per

@jtomson
Copy link
Contributor Author

jtomson commented Feb 14, 2022

Would it be OK to make the max log line size configurable, similar to rsyslog's MaxMessageSize and syslog-ng's log-msg-size? I would be happy to create a PR

@CamJN
Copy link
Member

CamJN commented Feb 15, 2022

I'm not against it in theory, so I'd be open to a PR. Make sure to sign the contributor agreement if you haven't before. https://www.phusion.nl/contributor

@tsu-shiuan
Copy link

@jtomson 👋 I'm running into this issue as well. I'm going to go ahead and create a PR unless objections :)

@jtomson
Copy link
Contributor Author

jtomson commented Aug 30, 2022

@tsu-shiuan yes by all means go ahead :) this fell off my radar

@tsu-shiuan
Copy link

@CamJN @jtomson I've submitted a pull request here :)

#2442

@jtomson
Copy link
Contributor Author

jtomson commented Jan 16, 2023

@CamJN please take a look at the newer PR #2458 since this seems to have stalled? thanks!

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