Skip to content

Replace papertrail with onlylogs - #458

Open
coorasse wants to merge 1 commit into
mainfrom
feature/drop_papertrail
Open

coorasse wants to merge 1 commit into
mainfrom
feature/drop_papertrail

Conversation

@coorasse

Copy link
Copy Markdown
Member

No description provided.

@sislr sislr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thank you!

Comment thread ruby_on_rails/onlylogs.md
Comment on lines +6 to +7
2. If the PaaS in use supports log drains, you'll find instructions directly on the project page you just created.
1. Otherwise, set `ONLYLOGS_DRAIN_URL` as an ENV variable on the application server.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd make these two comments more specific to our default = Deploio.

Comment thread ruby_on_rails/onlylogs.md
```ruby
# config/environments/production.rb
config.logger = Onlylogs::Logger.new(Rails.root.join("log", "production.log"))
config.logger.formatter.denylist = [/health_check/, /ping/]

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
config.logger.formatter.denylist = [/health_check/, /ping/]
config.logger.formatter.denylist = [/up/]

Matches the default Rails health endpoint. Why /ping/?
How do you ignore the asset requests?

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.

2 participants