diff --git a/lib/Raven/Client.php b/lib/Raven/Client.php index 7d155fffd..f9887b213 100644 --- a/lib/Raven/Client.php +++ b/lib/Raven/Client.php @@ -881,6 +881,9 @@ public function capture($data, $stack = null, $vars = null) if (empty($data['request'])) { unset($data['request']); } + if (empty($data['site'])) { + unset($data['site']); + } if (!$this->breadcrumbs->is_empty()) { $data['breadcrumbs'] = $this->breadcrumbs->fetch(); diff --git a/test/Raven/Tests/ClientTest.php b/test/Raven/Tests/ClientTest.php index 0c63e189f..b8fb70bf9 100644 --- a/test/Raven/Tests/ClientTest.php +++ b/test/Raven/Tests/ClientTest.php @@ -760,6 +760,21 @@ public function testDefaultProcessorsContainSanitizeDataProcessor() } /** + * @covers Raven_Client::capture + */ + public function testEmptySiteGetsRemoved() + { + $client = new Dummy_Raven_Client(); + $client->site = ''; + + $client->captureMessage("My message"); + $events = $client->getSentEvents(); + $this->assertSame(1, count($events)); + $event = array_pop($events); + $this->assertFalse(array_key_exists('site', $event)); + } + + /** * @covers Raven_Client::__construct * @covers Raven_Client::get_default_data */