search

Fullscript / yabeda-activejob

Yabeda Instrumentation for some metrics around rails activejobs
MIT License
8 stars 2 forks source link

Fix job_latency to work with Rails 7.2 #14

Closed mattbnz closed 2 weeks ago

mattbnz commented 3 months ago

Time.parse does not work for the unix timestamps (in seconds) now returned by Rails for Event.end.

Fixes: Fullscript/yabeda-activejob#13

padde commented 1 month ago

We've also encountered this during a dependabot update for Rails. Can we get this released? For now, we've monkey patched it.

andrewmarkle commented 2 weeks ago

Thanks @mattbnz. We merged in this solution which should work for Rails 7.1 and 7.2 https://github.com/Fullscript/yabeda-activejob/pull/12

It's on main now and working on a release.