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

Rails tagged log support #8

Closed
wants to merge 3 commits into from
Closed

Conversation

mitnal
Copy link

@mitnal mitnal commented May 25, 2012

Hi,

I just added Rails tagged logging support (via config.log_tags / default_options['tags']), similar to ActiveSupport::TaggedLogging.

The Tags are added to the hash as custom gelf message ('_tag_name' => <tag>).

Example usage:

config.log_tags = [:uuid, :remote_ip]
config.logger = GELF::Logger.new("localhost", 12201, 'LAN', {
    tags: [:some_name_for_uuid, :some_name_for_remote_ip] # same order as config.log_tags !
  })

@bryanl
Copy link

bryanl commented Jul 26, 2012

Is this dead?

@mitnal
Copy link
Author

mitnal commented Jul 26, 2012

Good question. I am still waiting for a response.

At the moment I am using my own fork in a project and without any problems.

@gravis
Copy link

gravis commented Aug 22, 2014

Would be so nice to have that merged!

@gravis
Copy link

gravis commented Jan 9, 2015

Any news on this?
Thanks

@andyjeffries
Copy link

+1 would love to have this in.

@joschi
Copy link
Contributor

joschi commented May 6, 2015

@mitnal Thanks for contributing to this project! To continue merging this PR, we need you to sign our Individual Contributor Assignment Agreement as described in https://www.graylog.org/contributing-to-graylog/

@mitnal
Copy link
Author

mitnal commented May 6, 2015

@joschi Sorry but I do not feel comfortable signing anything I do not fully understand and so I will not sign the CAA. And honestly I do not have time or patience to try to understand any kind of CAA.

But if anybody wants to base (even copy/paste) her/his pull request on my changes feel free to do so.

@mitnal mitnal closed this May 6, 2015
@markglenn
Copy link
Contributor

@mitnal I'll create a new pull request based on this code when I get some time next week. I've already signed the CAA. Not having tagged logs in our app has made it difficult for us to really debug properly, so this has been a pain point for us for a while.

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

Successfully merging this pull request may close these issues.

6 participants