Add local variables to notifications

honeybadger/payload.py

@@ -2,6 +2,7 @@
 import traceback
 import os
 import logging
+import inspect
 from six.moves import range
 from six.moves import zip
 from io import open
@@ -107,6 +108,12 @@ def create_payload(exception, exc_traceback=None, config=None, context=None, fin
     if exc_traceback is None:
         exc_traceback = sys.exc_info()[2]
 
+    # try and get local variables from stacktrace 
+    try:
+        local_variables = inspect.trace()[-1][0].f_locals
+    except Exception:
+        local_variables = None
+
     #if context is None, Initialize as an emptty dict
     if not context:
         context = {}
@@ -119,7 +126,10 @@ def create_payload(exception, exc_traceback=None, config=None, context=None, fin
         },
         'error':  error_payload(exception, exc_traceback, config, fingerprint),
         'server': server_payload(config),
-        'request': {'context': context}
+        'request': {
+            'context': context,
+            'local_variables': local_variables
+        }
     }
 
     return default_plugin_manager.generate_payload(payload, config, context)

honeybadger/tests/test_payload.py

@@ -4,12 +4,15 @@
 import os
 import sys
 
-from honeybadger.payload import error_payload
-from honeybadger.payload import server_payload
+from honeybadger.payload import (
+    create_payload, 
+    error_payload, 
+    server_payload
+)
 from honeybadger.config import Configuration
 
 from mock import patch
-from nose.tools import eq_, ok_
+from nose.tools import eq_, ok_, assert_raises
 
 # TODO: figure out how to run Django tests?
 
@@ -123,3 +126,17 @@ def test_psutil_is_optional():
     with patch.dict(sys.modules, {'psutil':None}):
         payload = server_payload(config)
         eq_(payload['stats'], {})
+
+def test_create_payload_with_local_variables():
+    config = Configuration()
+    with assert_raises(Exception):
+        test_local_variable = {"test": "var"}
+        exception = Exception('Test')
+        payload = create_payload(exception, config=config)
+        eq_(payload['request']['local_variables'], test_local_variable)
+
+def test_create_payload_without_local_cvariables():
+    config = Configuration()
+    exception = Exception('Test')
+    payload = create_payload(exception, config=config)
+    eq_(payload['request']['local_variables'], None)