-
Notifications
You must be signed in to change notification settings - Fork 13
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
Add logstash and lograge to the Rails app #691
Conversation
@@ -4,5 +4,6 @@ | |||
# Use this to limit dissemination of sensitive information. | |||
# See the ActiveSupport::ParameterFilter documentation for supported notations and behaviors. | |||
Rails.application.config.filter_parameters += [ | |||
:password, :passw, :email, :secret, :token, :_key, :crypt, :salt, :certificate, :otp, :ssn | |||
:password, :passw, :email, :secret, :token, :_key, :crypt, :salt, :certificate, :otp, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice
@@ -0,0 +1,34 @@ | |||
Rails.application.configure do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for adding all the comments :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. I checked out the log on stage. Looks good.
Fixes #682
Lograge allow us to log to a file in the logstash format AND retain the existing Rails logs. This will be helpful in the event that we need to do a deep dive into them to investigate an issue.
Deployed and running in stage. The new logs can be reviewed in the
/dmp/apps/dmptool/shared/log/lograge_stage.log
file