smailer 0.8.1 → 0.8.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 6bf1b2eaf8471b07434e71803ce46eb6d971ad81
-  data.tar.gz: 3c761ccfcf13f68a90d57590c541e3935b6895dd
+  metadata.gz: 597184be6e340a90a5aec0c7d877ebaeae878071
+  data.tar.gz: be0274a57ca1f64b01398ca45b2bab0ae770aba5
 SHA512:
-  metadata.gz: 6b98a822ebfcce95f8ece649ed0b99a3522dc8e2c924e7fa8faf8f55f706bc8d97cc55d15e54b5b83c13877f07dcb0393ec8b5437644137d07721b40a0d3389f
-  data.tar.gz: 58cde2ebfc9c01c7d2d5d36655e50143f815157a9c72c51ef21c98318081f1d061d9c8be48474bab319d1787624f2742e72f54ea3be4e03870259c357d128294
+  metadata.gz: 34a053d61e746507658a348e6f2f59a0d82a49b5e7ddae10648f5f9399f5a24fa7d9a3fc4c6e09db24beac000bcea155d8a9c62a70851d898377667c3711c779
+  data.tar.gz: 9747adc90ed4c575516df92f07aae7edc7a943b882d34e50d66c6cf1e0891720a252f1eeeb0cef4736501475b350936092126c12b063a086f5120f8330f8ae06

data/.rspec

@@ -0,0 +1 @@
+--colour

data/CHANGELOG.md

@@ -1,11 +1,24 @@
+## Version v0.8.2
+
+- Fixes broken unsubscribe and tracking links (broken by improper unique
+  message key regeneration). The issue was introduced in v0.8.0.
+
+No special actions are required to upgrade from v0.8.1.
+
 ## Version v0.8.1
 
 - Adds support for 'reply-to'
 
 **Major Bug Fixes:**
 
-- `QueuedMail.new(to: …, from: …, mail_campaign: …)` will work as expected 9ff2d80
-- `QueuedMail#body_text=` now works as expected
+- `QueuedMail.new(to: …, from: …, mail_campaign: …)` will work as expected [9ff2d80](https://github.com/livebg/smailer/commit/9ff2d80).
+- `QueuedMail#body_text=` now works as expected.
+
+**Upgrading from v0.8.0 to v0.8.1**
+
+To make the required changes in the database, you can use this
+[smailer_v0_8_0_to_v0_8_1 migration](upgrading/migrations/smailer_v0_8_0_to_v0_8_1.rb).
+You may need to alter this migration to fit your data.
 
 ## Version v0.8.0
 

data/lib/smailer/models/queued_mail.rb

@@ -94,7 +94,7 @@ module Smailer
       end
 
       def initialize_message_key
-        self.key = Digest::MD5.hexdigest("#{mail_campaign_id}, #{to}, #{SecureRandom.hex(15)} and #{id} compose this key.")
+        self.key = self[:key] || Digest::MD5.hexdigest("#{mail_campaign_id}, #{to}, #{SecureRandom.hex(15)} compose this key.")
       end
 
       def interpolate(text)

data/lib/smailer/version.rb

@@ -1,7 +1,7 @@
 module Smailer
   MAJOR = 0
   MINOR = 8
-  PATCH = 1
+  PATCH = 2
 
   VERSION = [MAJOR, MINOR, PATCH].join('.')
 end

data/spec/models/queued_mail_spec.rb

@@ -150,6 +150,22 @@ describe Smailer::Models::QueuedMail do
 
       expect(queued_mail_1.key).to_not eq(queued_mail_2.key)
     end
+
+    it 'remains the same once initialized' do
+      queued_mail = Smailer::Models::QueuedMail.new
+      queued_mail.mail_campaign_id = 42
+      queued_mail.to = 'test@example.com'
+
+      expect(SecureRandom).to receive(:hex).once.with(15).and_return('random-string')
+      expect(Digest::MD5).to receive(:hexdigest).once.with('42, test@example.com, random-string compose this key.').and_return('message-key')
+
+      expect(queued_mail.key).to eq('message-key')
+
+      queued_mail.save!
+      queued_mail.reload
+
+      expect(queued_mail.key).to eq('message-key')
+    end
   end
 
   describe '#require_uniqueness' do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: smailer
 version: !ruby/object:Gem::Version
-  version: 0.8.1
+  version: 0.8.2
 platform: ruby
 authors:
 - Dimitar Dimitrov
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-07-06 00:00:00.000000000 Z
+date: 2015-08-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -61,6 +61,7 @@ extensions: []
 extra_rdoc_files: []
 files:
 - ".gitignore"
+- ".rspec"
 - CHANGELOG.md
 - Gemfile
 - Gemfile.lock