Merge pull request #756 from xymbol/append-user-addr

Append userAddr to context

lib/airbrake/rack/context_filter.rb

@@ -29,6 +29,7 @@ def call(notice)
         context = notice[:context]
 
         context[:url] = request.url
+        context[:userAddr] = request.ip
         context[:userAgent] = request.user_agent
 
         add_framework_version(context)

spec/unit/rack/context_filter_spec.rb

@@ -44,6 +44,28 @@ def env_for(url, opts = {})
     end
   end
 
+  context "when visitor address is present" do
+    let(:opts) do
+      { 'REMOTE_ADDR' => '1.2.3.4' }
+    end
+
+    it "adds userAddr to the context" do
+      subject.call(notice)
+      expect(notice[:context][:userAddr]).to eq('1.2.3.4')
+    end
+  end
+
+  context "when visitor is behind a proxy or load balancer" do
+    let(:opts) do
+      { 'HTTP_X_FORWARDED_FOR' => '8.8.8.8, 9.9.9.9' }
+    end
+
+    it "adds userAddr to the context" do
+      subject.call(notice)
+      expect(notice[:context][:userAddr]).to eq('9.9.9.9')
+    end
+  end
+
   context "when controller is present" do
     let(:controller) do
       double.tap do |ctrl|