From 0f0d2183a1d11648dc523d5af714a659fe01189b Mon Sep 17 00:00:00 2001
From: Ivan Socolsky <jungans@gmail.com>
Date: Tue, 9 Jun 2015 16:40:46 -0300
Subject: [PATCH 1/2] test duplicate email

---
 test/integration/server.js | 37 +++++++++++++++++++++++++++++++++++++
 1 file changed, 37 insertions(+)

diff --git a/test/integration/server.js b/test/integration/server.js
index 7fb3695..74e13db 100644
--- a/test/integration/server.js
+++ b/test/integration/server.js
@@ -459,6 +459,43 @@ describe('Wallet service', function() {
         });
       });
     });
+
+    it('should notify each email address only once', function(done) {
+      // Set same email address for copayer1 and copayer2
+      server.savePreferences({
+        email: 'copayer2@domain.com',
+      }, function(err) {
+        server.createAddress({}, function(err, address) {
+          should.not.exist(err);
+
+          // Simulate incoming tx notification
+          server._notify('NewIncomingTx', {
+            txid: '999',
+            address: address,
+            amount: 12300000,
+          }, function(err) {
+            setTimeout(function() {
+              var calls = mailerStub.sendMail.getCalls();
+              calls.length.should.equal(2);
+              var emails = _.map(calls, function(c) {
+                return c.args[0];
+              });
+              _.difference(['copayer2@domain.com', 'copayer3@domain.com'], _.pluck(emails, 'to')).should.be.empty;
+              var one = emails[0];
+              one.from.should.equal('bws@dummy.net');
+              one.subject.should.contain('New payment received');
+              one.text.should.contain(wallet.name);
+              one.text.should.contain('123,000');
+              server.storage.fetchUnsentEmails(function(err, unsent) {
+                should.not.exist(err);
+                unsent.should.be.empty;
+                done();
+              });
+            }, 100);
+          });
+        });
+      });
+    });
   });
 
 

From 8151f68c3cd68d1723ffda83945ebc1385fcaae5 Mon Sep 17 00:00:00 2001
From: Ivan Socolsky <jungans@gmail.com>
Date: Tue, 9 Jun 2015 16:55:46 -0300
Subject: [PATCH 2/2] unique recipient list

---
 lib/emailservice.js | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/lib/emailservice.js b/lib/emailservice.js
index 9d10c68..e5ad0d8 100644
--- a/lib/emailservice.js
+++ b/lib/emailservice.js
@@ -124,8 +124,11 @@ EmailService.prototype._getRecipientsList = function(notification, emailType, cb
     if (err) return cb(err);
     if (_.isEmpty(preferences)) return cb(null, []);
 
+    var usedEmails = {};
     var recipients = _.compact(_.map(preferences, function(p) {
-      if (!p.email) return;
+      if (!p.email || usedEmails[p.email]) return;
+
+      usedEmails[p.email] = true;
       if (notification.creatorId == p.copayerId && !emailType.notifyDoer) return;
       return {
         copayerId: p.copayerId,