Merge pull request #78 from Sija/develop

v1.9

README.md

@@ -30,7 +30,7 @@ from folks at [@getsentry](https://github.com/getsentry).
 - [x] Interfaces (Message, Exception, Stacktrace, User, HTTP, ...)
 - [x] Contexts (user, tags, extra, os, runtime)
 - [x] Breadcrumbs
-- [x] Integrations ([Kemal](https://github.com/kemalcr/kemal), [Amber](https://github.com/amberframework/amber), [Lucky](https://github.com/luckyframework/lucky), [Sidekiq.cr](https://github.com/mperham/sidekiq.cr))
+- [x] Integrations ([Kemal](https://github.com/kemalcr/kemal), [Amber](https://github.com/amberframework/amber), [Lucky](https://github.com/luckyframework/lucky), [Sidekiq.cr](https://github.com/mperham/sidekiq.cr), [action-controller](https://github.com/spider-gazelle/action-controller))
 - [x] Async support
 - [x] User Feedback
 - [x] Source code context for stack traces

shard.yml

@@ -1,10 +1,13 @@
 name: raven
-version: 1.8.1
+version: 1.9.0
 
 authors:
   - Sijawusz Pur Rahnama <sija@sija.pl>
 
 dependencies:
+  backtracer:
+    github: Sija/backtracer.cr
+    version: ~> 1.2.0
   any_hash:
     github: Sija/any_hash.cr
     version: ~> 0.2.3

spec/raven/event_spec.cr

@@ -280,8 +280,8 @@ describe Raven::Event do
         frames[0][:lineno].should eq(1412)
         frames[0][:colno].should eq(1)
         frames[0][:function].should eq("other_function")
-        frames[0][:abs_path].should eq("some/relative/path")
-        frames[0][:filename].should eq(frames[0][:abs_path])
+        frames[0][:abs_path].should eq("#{Dir.current}/some/relative/path")
+        frames[0][:filename].should eq("some/relative/path")
         frames[0][:package].should be_nil
         frames[0][:in_app].should be_false
 

src/crash_handler.cr

@@ -25,7 +25,7 @@ module Raven
     # [0x105798128] main +40
     # ```
     CRYSTAL_CRASH_PATTERN =
-      /(?<message>[^\n]+)\n(?<backtrace>\[#{Backtrace::Line::ADDR_FORMAT}\] .*)$/m
+      /(?<message>[^\n]+)\n(?<backtrace>\[(?<addr>0x[a-f0-9]+)\] .*)$/im
 
     # Example:
     #
@@ -51,12 +51,7 @@ module Raven
     # An `Array` of arguments passed to process.
     property args : Array(String)?
 
-    # FIXME: doesn't work yet due to usage of global Raven within `Backtrace::Line`.
-    #
-    # ```
-    # getter raven : Instance { Instance.new }
-    # ```
-    getter raven : Instance { Raven.instance }
+    getter raven : Instance { Instance.new }
 
     delegate :context, :configuration, :configure, :capture,
       to: raven

src/raven.cr

@@ -1,3 +1,4 @@
+require "backtracer"
 require "any_hash"
 
 require "./raven/ext/*"