Forum: Ruby on Rails Send mail on Rails.logger.error

Hi,

I would like to receive an email, if anywhere in my application a
Rails.logger.errror method is triggered.

I know about all these exception_notfiers, but Rails.logger.error is
triggered without an exception as well in my application and these
exception_notfier gems will not catch them.

Would be great if anybody could offer me a solution.

Thanks a lot in advance.

--
Volker

On Sat, Jan 12, 2013 at 6:47 AM, vhochstein <vhochstein@googlemail.com> 
wrote:
> Hi,
>
> I would like to receive an email, if anywhere in my application a
> Rails.logger.errror method is triggered.
>
> I know about all these exception_notfiers, but Rails.logger.error is
> triggered without an exception as well in my application and these
> exception_notfier gems will not catch them.

This is indeed an issue. Currently, our app logs to the standard Rails
logger sometimes, and to Data Dog sometimes -- these require two
different calls, and it is sometimes confusing when to use which.

This shouldn't be too much of a problem, since you can specify a new
logger for Rails.logger to use (and use per environment, if wanted).
As for having an email option, log4r seems more useful.

On Jan 12, 1:47pm, vhochstein <vhochst...@googlemail.com> wrote:
> Hi,
>
> I would like to receive an email, if anywhere in my application a
> Rails.logger.errror method is triggered.
>
> I know about all these exception_notfiers, but Rails.logger.error is
> triggered without an exception as well in my application and these
> exception_notfier gems will not catch them.
>
> Would be great if anybody could offer me a solution.

We used to do this via syslog - our hosting provider had a shell
script that tailed the syslog file containing entries relative to our
app and would email us anything of that level or above.
I'm afraid that was with a previous job so I no longer have access to
that but it is definitely doable

Fred.

I don't really know that much about the interns of Rails.logger.error
method but how about overwriting it for your app?

use alias on the old method and do something like this:

alias :old_error :error

def error
   # send your mail or whatever
   old_error
end

would at least do its job without braking all the stuff the logger does.

Am Samstag, 12. Januar 2013 13:47:12 UTC+1 schrieb vhochstein:

On Sun, Jan 13, 2013 at 5:42 AM, Crispin Schffler
<crispinschaeffler@gmail.com> wrote:
>
> would at least do its job without braking all the stuff the logger does.

Uhm, you guys do know that you do not have to resort to such dirty
tactics?  Read:

Rails.logger.error do
  "There was an error".tap do |s|
    # Do mailing Work Here
  end
end

cat log/development.log
There was an error

Yeah, as I told, i don't know the insides of the error method or the
correct specification.. Just wanted to give a hint how you could do it 
to
get it to work quickly and without much trouble.

And in my opinion its not that dirty to alias a function if you know 
what
you do.
Sure you can use a block to do the mailing.



Am Sonntag, 13. Januar 2013 13:00:50 UTC+1 schrieb Jordon Bedwell:

Just wanted to give a hint that it's not that hard to make it work.

Your approach is much cleaner and should be used of course.

Am Sonntag, 13. Januar 2013 13:00:50 UTC+1 schrieb Jordon Bedwell:

On Sun, Jan 13, 2013 at 6:06 AM, Crispin Schffler
<crispinschaeffler@gmail.com> wrote:
> Yeah, as I told, i don't know the insides of the error method or the correct
> specification.. Just wanted to give a hint how you could do it to get it to
> work quickly and without much trouble.
>
> And in my opinion its not that dirty to alias a function if you know what
> you do.
> Sure you can use a block to do the mailing.

Until Joe down the street decides he wants to be as clever as you and
alias it to old_method and erase yours accidently, or until Marline
from up the street asks you why there are objects hanging around when
you mean to replace them.  If you intend for :old_method to be an
object on it's parent then great more power to you, if you don't then
unbind the method and use define_method and be cleaner in your source
even if you are already being dirty by monkey patching.