Trim http data and bump limit to 512

dcramer · dcramer · commit 472c6c8ea405 · 2013-06-07T12:44:21.000-07:00
diff --git a/docs/developer/client/index.rst b/docs/developer/client/index.rst
@@ -556,7 +556,7 @@ Most arbitrary values in Sentry have their size restricted. This means any
 values that are sent as metadata (such as variables in a stacktrace) as well
 as things like extra data, or tags.
 
-- Unstructured data is limited to 256 characters.
+- Unstructured data is limited to 512 characters.
 - Event IDs are limited to 32 characters.
 - Tag keys are limited to 32 characters.
 - Tag values are limited to 200 characters.
diff --git a/src/sentry/constants.py b/src/sentry/constants.py
@@ -156,4 +156,4 @@
 
 # Prevent variables (e.g. context locals, extra data) from exceeding this size
 # in bytesish
-MAX_VARIABLE_SIZE = 256
+MAX_VARIABLE_SIZE = 512
diff --git a/src/sentry/manager.py b/src/sentry/manager.py
@@ -480,6 +480,13 @@ def save_data(self, project, data):
                 if value:
                     exc_data[key] = trim(value)
 
+        if 'sentry.interfaces.Http' in data:
+            http_data = data['sentry.interfaces.Http']
+            for key in ('data', 'cookies', 'querystring', 'headers', 'env'):
+                value = http_data.get(key)
+                if value:
+                    http_data[key] = trim(value)
+
         kwargs = {
             'level': level,
             'message': message,
