Skip to content

Commit 0338b9d

Browse files
authored
ref: Simplify limits from the integrations (#39)
1 parent 45a10c0 commit 0338b9d

File tree

3 files changed

+8
-8
lines changed

3 files changed

+8
-8
lines changed

sentry_sdk/integrations/_wsgi.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ def extract_into_event(self, event):
7474
data = AnnotatedValue(
7575
"",
7676
{
77-
"rem": [["!rawbody", "x", 0, content_length]],
77+
"rem": [["!raw", "x", 0, content_length]],
7878
"len": content_length,
7979
},
8080
)
@@ -109,7 +109,7 @@ def parsed_body(self):
109109
for k, v in files.items():
110110
size = self.size_of_file(v)
111111
data[k] = AnnotatedValue(
112-
"", {"len": size, "rem": [["!filecontent", "x", 0, size]]}
112+
"", {"len": size, "rem": [["!raw", "x", 0, size]]}
113113
)
114114

115115
return data

sentry_sdk/utils.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -493,7 +493,7 @@ def convert_types(obj):
493493
def strip_databag(obj, remaining_depth=20):
494494
assert not isinstance(obj, bytes), "bytes should have been normalized before"
495495
if remaining_depth <= 0:
496-
return AnnotatedValue(None, {"rem": [["!dep", "x"]]})
496+
return AnnotatedValue(None, {"rem": [["!limit", "x"]]})
497497
if isinstance(obj, text_type):
498498
return strip_string(obj)
499499
if isinstance(obj, Mapping):
@@ -515,7 +515,7 @@ def strip_string(value, assume_length=None, max_length=512):
515515
value=value[: max_length - 3] + u"...",
516516
metadata={
517517
"len": assume_length,
518-
"rem": [["!len", "x", max_length - 3, max_length]],
518+
"rem": [["!limit", "x", max_length - 3, max_length]],
519519
},
520520
)
521521
return value[:max_length]

tests/integrations/flask/test_flask.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -179,7 +179,7 @@ def index():
179179

180180
event, = events
181181
assert event[""]["request"]["data"]["foo"]["bar"] == {
182-
"": {"len": 2000, "rem": [["!len", "x", 509, 512]]}
182+
"": {"len": 2000, "rem": [["!limit", "x", 509, 512]]}
183183
}
184184
assert len(event["request"]["data"]["foo"]["bar"]) == 512
185185

@@ -205,7 +205,7 @@ def index():
205205

206206
event, = events
207207
assert event[""]["request"]["data"]["foo"] == {
208-
"": {"len": 2000, "rem": [["!len", "x", 509, 512]]}
208+
"": {"len": 2000, "rem": [["!limit", "x", 509, 512]]}
209209
}
210210
assert len(event["request"]["data"]["foo"]) == 512
211211

@@ -261,12 +261,12 @@ def index():
261261

262262
event, = events
263263
assert event[""]["request"]["data"]["foo"] == {
264-
"": {"len": 2000, "rem": [["!len", "x", 509, 512]]}
264+
"": {"len": 2000, "rem": [["!limit", "x", 509, 512]]}
265265
}
266266
assert len(event["request"]["data"]["foo"]) == 512
267267

268268
assert event[""]["request"]["data"]["file"] == {
269-
"": {"len": 0, "rem": [["!filecontent", "x", 0, 0]]}
269+
"": {"len": 0, "rem": [["!raw", "x", 0, 0]]}
270270
}
271271
assert not event["request"]["data"]["file"]
272272

0 commit comments

Comments
 (0)