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

Log to STDOUT when set through option #184

Closed
3 of 4 tasks
tombruijn opened this issue Oct 6, 2016 · 2 comments
Closed
3 of 4 tasks

Log to STDOUT when set through option #184

tombruijn opened this issue Oct 6, 2016 · 2 comments
Assignees

Comments

@tombruijn
Copy link
Member

tombruijn commented Oct 6, 2016

Let's give the option to the user instead of using a workaround by tricking the system into thinking it's on Heroku (the only time we log to stdout on purpose) DYNO=foo ruby app.rb

# config/appsignal.yml

log_path: ~
# ~ is nil

# or
log: stdout

log: stdout has my preference

When no path is writable it should log to stdout by default.

See also https://app.intercom.io/a/apps/yzor8gyw/inbox/conversation/6239874634


TODO

@tombruijn tombruijn self-assigned this Oct 6, 2016
tombruijn added a commit that referenced this issue Oct 13, 2016
Will silence any warnings of not writable log file paths when set to
`log: stdout`.

Part of #184
Doesn't fix it entirely as it only redirects the ruby gem's output.
@tombruijn
Copy link
Member Author

Last addition: output last lines from appsignal.log to STDOUT on agent connection failure, when the agent fails to start, added in gem v2.0.4.

Closing

@tonytonyjan
Copy link

I set log: stdout with v2.2.1, but it still writes to log/appsignal.log, having no idea how to diagnosis.

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