Add dataCallback option

Author: dcramer

README.md

@@ -198,6 +198,18 @@ client = new raven.Client('{{ SENTRY_DSN }}', {
 });
 ```
 
+## Pre-processing data
+Pass the `dataCallback` configuration value:
+
+```javascript
+client = new raven.Client('{{ SENTRY_DSN }}', {
+  dataCallback: function(data) {
+    delete data.request.env;
+    return data;
+  }
+});
+```
+
 ## Disable Raven
 Pass `false` as the DSN (or any falsey value).
 

lib/client.js

@@ -30,6 +30,7 @@ var Client = function Client(dsn, options) {
     this.transport = options.transport || transports[this.dsn.protocol];
 
     this.loggerName = options.logger || '';
+    this.dataCallback = options.dataCallback || null;
 
     // enabled if a dsn is set
     this._enabled = !!this.dsn;
@@ -61,6 +62,10 @@ _.process = function process(kwargs) {
 
     var ident = {'id': kwargs['event_id']};
 
+    if (this.dataCallback) {
+        kwargs = this.dataCallback(kwargs);
+    }
+
     // this will happen asynchronously. We don't care about it's response.
     this._enabled && this.send(kwargs, ident);
 

test/raven.client.js

@@ -297,6 +297,40 @@ describe('raven.Client', function(){
         });
     });
 
+    describe('#process()', function(){
+        it('should respect dataCallback', function(){
+            var scope = nock('https://app.getsentry.com')
+                .filteringRequestBody(/.*/, '*')
+                .post('/api/store/', '*')
+                .reply(200, function(uri, body) {
+                    zlib.inflate(new Buffer(body, 'base64'), function(err, dec) {
+                      if (err) return done(err);
+                      var msg = JSON.parse(dec.toString());
+                      var extra = msg.extra;
+
+                      extra.should.not.have.property('foo');
+                    });
+                    return 'OK';
+                });
+
+            var client = new raven.Client(dsn, {
+                dataCallback: function(data){
+                    delete data.extra.foo;
+                    return data;
+                }
+            });
+
+            client.process({
+                message: 'test',
+                extra: {foo: 'bar'}
+            });
+
+            client.on('logged', function(){
+                scope.done();
+            });
+        });
+    });
+
     it('should use a custom transport', function(){
         var expected = {
             protocol: 'udp',