Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Serializer.php #553

Merged
merged 10 commits into from
May 2, 2018
Merged

Update Serializer.php #553

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
e8f7209
Update Serializer.php
truechudo Feb 21, 2018
43596f6
Update SerializerTest.php
truechudo Mar 8, 2018
5a29c9a
Merge branch 'master' into truechudo-patch-1
stayallive May 2, 2018
4ca88df
Add test for clipping UTF-8 characters
stayallive May 2, 2018
377f233
Fix long string test not counting non-ascii characters correctly
stayallive May 2, 2018
f160746
Specify encoding and make sure the mbstring extension is loaded
stayallive May 2, 2018
bd2bd35
Only use printable ascii in long string serialize test
stayallive May 2, 2018
e3ee219
Fix testLongString tests (last time)
stayallive May 2, 2018
3a23415
Increase memory limit for regression test
stayallive May 2, 2018
f032048
Merge branch 'master' into truechudo-patch-1
stayallive May 2, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion lib/Raven/Serializer.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ protected function serializeString($value)
}

if (strlen($value) > 1024) {
$value = substr($value, 0, 1014) . ' {clipped}';
$value = mb_substr($value, 0, 1014) . ' {clipped}';
}

return $value;
Expand Down
2 changes: 1 addition & 1 deletion test/Raven/Tests/SerializerTest.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -126,7 +126,7 @@ public function testLongString()
}
$result = $serializer->serialize($input);
$this->assertInternalType('string', $result);
$this->assertLessThanOrEqual(1024, strlen($result));
$this->assertLessThanOrEqual(1024, mb_strlen($result));
}
}
}
Expand Down