Merge pull request #102 from getsentry/http-context

set http context

Author: mitchellhenke

lib/sentry/context.ex

@@ -15,6 +15,7 @@ defmodule Sentry.Context do
   @user_key :user
   @tags_key :tags
   @extra_key :extra
+  @request_key :request
   @breadcrumbs_key :breadcrumbs
 
   def get_all do
@@ -23,6 +24,7 @@ defmodule Sentry.Context do
       user: Map.get(context, @user_key, %{}),
       tags: Map.get(context, @tags_key, %{}),
       extra: Map.get(context, @extra_key, %{}),
+      request: Map.get(context, @request_key, %{}),
       breadcrumbs: Map.get(context, @breadcrumbs_key, []),
     }
   end
@@ -42,6 +44,11 @@ defmodule Sentry.Context do
     |> set_context(@tags_key, map)
   end
 
+  def set_http_context(map) when is_map(map) do
+    get_context()
+    |> set_context(@request_key, map)
+  end
+
   def clear_all do
     Process.delete(@process_dictionary_key)
   end

lib/sentry/event.ex

@@ -41,7 +41,8 @@ defmodule Sentry.Event do
     %{user: user_context,
      tags: tags_context,
      extra: extra_context,
-     breadcrumbs: breadcrumbs_context} = Sentry.Context.get_all()
+     breadcrumbs: breadcrumbs_context,
+     request: request_context} = Sentry.Context.get_all()
 
     stacktrace = Keyword.get(opts, :stacktrace, [])
 
@@ -52,7 +53,8 @@ defmodule Sentry.Event do
     tags = Application.get_env(:sentry, :tags, %{})
             |> Map.merge(tags_context)
             |> Map.merge(Keyword.get(opts, :tags, %{}))
-    request = Keyword.get(opts, :request, %{})
+    request = request_context
+              |> Map.merge(Keyword.get(opts, :request, %{}))
     breadcrumbs = Keyword.get(opts, :breadcrumbs, [])
                   |> Kernel.++(breadcrumbs_context)
 

test/context_test.exs

@@ -54,6 +54,15 @@ defmodule Sentry.ContextTest do
     assert event.user == %{}
   end
 
+  test "storing http context appears when generating event" do
+    Sentry.Context.set_http_context(%{"url" => "https://wow"})
+
+    exception = RuntimeError.exception("error")
+    event = Sentry.Event.transform_exception(exception, [request: %{"method" => "GET"}])
+
+    assert event.request == %{"url" => "https://wow", "method" => "GET"}
+  end
+
   test "passing in extra context as option overrides Sentry.Context" do
     Sentry.Context.set_extra_context(%{"key" => "345", "key1" => "123"})