rpush 1.0.0-java → 2.0.0-java

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: dd3933018cb97f85a06df0df090aade180c8763c
-  data.tar.gz: 689b13f47bb1ec8f2a371ad175e63e5c929ba6b6
+  metadata.gz: 61cf526e008ad5aba0d3523150428719f09a85a3
+  data.tar.gz: c126e4d03cefc7fe33e99b5b49f6f2854cfcabfd
 SHA512:
-  metadata.gz: 5ca7768f9249d0f04553bd505f5f4bf929538a730eb24ce59bc37f041fddde8a55588e8758ce313902bfcd32716a49d51acb85ee226bb7e4343d0127b2922249
-  data.tar.gz: d5cbc3465122d36dd5222928fea8ab901a77d58bffdf6d25c84092ea43c8cbeac4b9f004241caabbc5bb8f27fc112dc22dba2d0be2f7e93fba7937119a1739d1
+  metadata.gz: 0a871d37518b8643b373c31186bd97242eb23fb44d524c4c1c2d23bd002bf4d49020ee6883b37864613f5a2c85dc47561676ed06c9e3997342c52e4ec59db287
+  data.tar.gz: 98529b49e823bb026644a6b1b299ce7e0ffa4a373c47cb70f7fc7484900febdc42c82603d541624a2bcd9e3d8a2622c815489db8f1b510ff95e8f2e7af401e1f

data/CHANGELOG.md

@@ -1,3 +1,18 @@
+## 2.0.0 (Sept 6, 2014)
+  * Use APNs enhanced binary format version 2.
+  * Support running multiple Rpush processes when using ActiveRecord and Redis.
+  * APNs error detection is now performed asynchronously, 'check_for_errors' is therefore deprecated.
+  * Deprecated attributes_for_device accessors. Use data instead.
+  * Fix signal handling to work with Ruby 2.x. (#40).
+  * You no longer need to signal HUP after creating a new app, they will be loaded automatically for you.
+  * APNs notifications are now delivered in batches, greatly improving throughput.
+  * Signaling HUP now also causes Rpush to immediately check for new notifications.
+  * The 'wakeup' config option has been removed.
+  * The 'batch_storage_updates' config option has been deprecated, storage backends will now always batch updates where appropriate.
+  * The rpush process title updates with number of queued notifications and number of dispatchers.
+  * Rpush::Apns::Feedback#app has been renamed to app_id and is now an Integer.
+  * An app is restarted when the HUP signal is received if its certificate or environment attribute changed.
+
 ## 1.0.0 (Feb 9, 2014)
   * Renamed to Rpush (from Rapns). Version number reset to 1.0.0.
   * Reduce default batch size to 100.

data/README.md

@@ -1,24 +1,32 @@
 [![Build Status](https://secure.travis-ci.org/rpush/rpush.png?branch=master)](http://travis-ci.org/rpush/rpush)
 [![Code Climate](https://codeclimate.com/github/rpush/rpush.png)](https://codeclimate.com/github/rpush/rpush)
-[![Coverage Status](https://coveralls.io/repos/rpush/rpush/badge.png?branch=master)](https://coveralls.io/r/rpush/rpush?branch=master)
+[![Code Coverage](https://codeclimate.com/github/rpush/rpush/coverage.png)](https://codeclimate.com/github/rpush/rpush)
 [![Gem Version](https://badge.fury.io/rb/rpush.png)](http://badge.fury.io/rb/rpush)
 
-<img src="https://raw.github.com/rpush/rpush/master/rpush.png" align="right" width="200px" />
+<img src="https://raw.github.com/rpush/rpush/master/logo.png" align="right" width="200px" />
 
 ### Rpush. The push notification service for Ruby.
 
-* Supports:
+* Supported services:
   * **Apple Push Notification Service**
-  * **Google Cloud Messaging)**
-  * **Amazon Devide Messaging**
-  * **Windows Push Notification Service**.
+  * **Google Cloud Messaging**
+  * **Amazon Device Messaging**
+  * **Windows Phone Push Notification Service**
+
+
+* Supported storage backends:
+  * ActiveRecord
+  * Redis
+  * More coming soon!
+
+
 * Seamless Rails (3, 4) integration.
-* Scalable - choose the number of persistent connections for each app.
-* Designed for uptime - signal -HUP to add, update apps.
-* Run as a daemon or inside an [existing processs](https://github.com/rpush/rpush/wiki/Embedding-API).
+* Scales vertically (threading) and horizontally (multiple processes).
+* Designed for uptime - new apps are loaded automatically, signal `HUP` to update running apps.
+* Run as a daemon or inside an [existing process](https://github.com/rpush/rpush/wiki/Embedding-API).
 * Use in a scheduler for low-workload deployments ([Push API](https://github.com/rpush/rpush/wiki/Push-API)).
 * Hooks for fine-grained instrumentation and error handling ([Reflection API](https://github.com/rpush/rpush/wiki/Reflection-API)).
-* Works with MRI, JRuby, Rubinius 1.9, 2.0, 2.1.
+* Works with MRI, JRuby and Rubinius.
 
 
 ### Getting Started
@@ -29,7 +37,7 @@ Add it to your Gemfile:
 gem 'rpush'
 ```
 
-Generate the migrations, rpush.yml and migrate:
+Generate the migrations, rpush.rb and migrate:
 
 ```
 rails g rpush
@@ -57,7 +65,7 @@ n = Rpush::Apns::Notification.new
 n.app = Rpush::Apns::App.find_by_name("ios_app")
 n.device_token = "..."
 n.alert = "hi mom!"
-n.attributes_for_device = {:foo => :bar}
+n.data = { foo: :bar }
 n.save!
 ```
 
@@ -76,8 +84,8 @@ app.save!
 ```ruby
 n = Rpush::Gcm::Notification.new
 n.app = Rpush::Gcm::App.find_by_name("android_app")
-n.registration_ids = ["..."]
-n.data = {:message => "hi mom!"}
+n.registration_ids = ["token", "..."]
+n.data = { message: "hi mom!" }
 n.save!
 ```
 
@@ -98,7 +106,7 @@ app.save!
 n = Rpush::Adm::Notification.new
 n.app = Rpush::Adm::App.find_by_name("kindle_app")
 n.registration_ids = ["..."]
-n.data = {:message => "hi mom!"}
+n.data = { message: "hi mom!"}
 n.collapse_key = "Optional consolidationKey"
 n.save!
 ```
@@ -122,33 +130,40 @@ n.alert = "..."
 n.save!
 ```
 
-### Starting Rpush
+### Running Rpush
+
+It is recommended to run Rpush as a separate process in most cases, though embedding and manual modes are provided for low-workload environments.
 
-As a daemon:
+#### As a daemon (recommended):
 
     cd /path/to/rails/app
     rpush <Rails environment> [options]
 
-Inside an existing process (see [Embedding API](https://github.com/rpush/rpush/wiki/Embedding-API)):
+#### Embedded inside an existing process
 
 ```ruby
+# Call this during startup of your application, for example, by adding it to the end of config/initializers/rpush.rb
 Rpush.embed
 ```
 
-*Please note that only ever a single instance of Rpush should be running.*
+See [Embedding API](https://github.com/rpush/rpush/wiki/Embedding-API) for more details.
 
-In a scheduler (see [Push API](https://github.com/rpush/rpush/wiki/Push-API)):
+#### Manually (in a scheduler)
 
 ```ruby
 Rpush.push
 Rpush.apns_feedback
 ```
 
+See [Push API](https://github.com/rpush/rpush/wiki/Push-API) for more details.
+
+### Configuration
+
 See [Configuration](https://github.com/rpush/rpush/wiki/Configuration) for a list of options, or run `rpush --help`.
 
 ### Updating Rpush
 
-After updating you should run `rails g rpush` to check for any new migrations.
+If you're using ActiveRecord, you should run `rails g rpush` after upgrading Rpush to check for any new migrations.
 
 ### Wiki
 
@@ -172,7 +187,7 @@ After updating you should run `rails g rpush` to check for any new migrations.
 * [Silent failures might be dropped connections](https://github.com/rpush/rpush/wiki/Dropped-connections)
 
 ### Google Cloud Messaging
-* [Notification Options](https://github.com/rpush/rpush/wiki//GCM-Notification-Options)
+* [Notification Options](https://github.com/rpush/rpush/wiki/GCM-Notification-Options)
 * [Canonical IDs](https://github.com/rpush/rpush/wiki/Canonical-IDs)
 * [Delivery Failures & Retries](https://github.com/rpush/rpush/wiki/Delivery-Failures-&-Retries)
 

data/bin/rpush

@@ -12,12 +12,21 @@ options = ARGV.options do |opts|
   opts.on('-f', '--foreground', 'Run in the foreground.') { config.foreground = true }
   opts.on('-P N', '--db-poll N', Integer, "Frequency in seconds to check for new notifications.") { |n| config.push_poll = n }
   opts.on('-F N', '--feedback-poll N', Integer, "Frequency in seconds to check for feedback.") { |n| config.feedback_poll = n }
-  opts.on('-e', '--no-error-checks', 'Disable APNs error checking after notification delivery.') { config.check_for_errors = false }
+  opts.on('-e', '--no-error-checks', 'Disable APNs error checking after notification delivery.') { config.check_for_errors = false } # deprecated
   opts.on('-p PATH', '--pid-file PATH', String, 'Path to write PID file. Relative to Rails root unless absolute.') { |path| config.pid_file = path }
   opts.on('-b N', '--batch-size N', Integer, 'Storage backend notification batch size.') { |n| config.batch_size = n }
-  opts.on('-B', '--[no-]batch-storage-updates', 'Perform storage updates in batches.') { |v| config.batch_storage_updates = v }
-  opts.on('-v', '--version', 'Print the version.') { puts "rpush #{Rpush::VERSION}"; exit }
-  opts.on('-h', '--help', 'You\'re looking at it.') { puts opts; exit }
+  opts.on('-B', '--[no-]batch-storage-updates', 'Perform storage updates in batches.') { |v| config.batch_storage_updates = v } # deprecated
+  opts.on('-v', '--version', 'Print the version.') do
+    puts "rpush #{Rpush::VERSION}"
+    exit
+  end
+  opts.on('-h', '--help', 'You\'re looking at it.') do
+    hidden_switches = %w(-e --no-error-checks)
+    puts opts.to_s.split("\n").delete_if do |line|
+      hidden_switches.any? { |s| line =~ /#{s}/ }
+    end.join("\n")
+    exit
+  end
 end
 
 if environment.nil? || environment =~ /^-/

data/lib/generators/rpush_generator.rb

@@ -23,6 +23,8 @@ class RpushGenerator < Rails::Generators::Base
     else
       add_rpush_migration('add_rpush')
     end
+
+    add_rpush_migration('rpush_2_0_0_updates')
   end
 
    def copy_config

data/lib/generators/templates/add_adm.rb

@@ -6,14 +6,14 @@ class AddAdm < ActiveRecord::Migration
   end
 
   def self.up
-    add_column :rapns_apps, :client_id, :string, :null => true
-    add_column :rapns_apps, :client_secret, :string, :null => true
-    add_column :rapns_apps, :access_token, :string, :null => true
-    add_column :rapns_apps, :access_token_expiration, :datetime, :null => true
+    add_column :rapns_apps, :client_id, :string, null: true
+    add_column :rapns_apps, :client_secret, :string, null: true
+    add_column :rapns_apps, :access_token, :string, null: true
+    add_column :rapns_apps, :access_token_expiration, :datetime, null: true
   end
 
   def self.down
-    AddAdm::Rapns::Notification.where(:type => 'Rapns::Adm::Notification').delete_all
+    AddAdm::Rapns::Notification.where(type: 'Rapns::Adm::Notification').delete_all
 
     remove_column :rapns_apps, :client_id
     remove_column :rapns_apps, :client_secret

data/lib/generators/templates/add_alert_is_json_to_rapns_notifications.rb

@@ -1,6 +1,6 @@
 class AddAlertIsJsonToRapnsNotifications < ActiveRecord::Migration
   def self.up
-    add_column :rapns_notifications, :alert_is_json, :boolean, :null => true, :default => false
+    add_column :rapns_notifications, :alert_is_json, :boolean, null: true, default: false
   end
 
   def self.down

data/lib/generators/templates/add_app_to_rapns.rb

@@ -1,7 +1,7 @@
 class AddAppToRapns < ActiveRecord::Migration
   def self.up
-    add_column :rapns_notifications, :app, :string, :null => true
-    add_column :rapns_feedback, :app, :string, :null => true
+    add_column :rapns_notifications, :app, :string, null: true
+    add_column :rapns_feedback, :app, :string, null: true
   end
 
   def self.down

data/lib/generators/templates/add_fail_after_to_rpush_notifications.rb

@@ -1,6 +1,6 @@
 class AddFailAfterToRpushNotifications < ActiveRecord::Migration
   def self.up
-    add_column :rpush_notifications, :fail_after, :timestamp, :null => true
+    add_column :rpush_notifications, :fail_after, :timestamp, null: true
   end
 
   def self.down

data/lib/generators/templates/add_gcm.rb

@@ -11,34 +11,34 @@ class AddGcm < ActiveRecord::Migration
   end
 
   def self.up
-    add_column :rapns_notifications, :type, :string, :null => true
-    add_column :rapns_apps, :type, :string, :null => true
+    add_column :rapns_notifications, :type, :string, null: true
+    add_column :rapns_apps, :type, :string, null: true
 
-    AddGcm::Rapns::Notification.update_all :type => 'Rapns::Apns::Notification'
-    AddGcm::Rapns::App.update_all :type => 'Rapns::Apns::App'
+    AddGcm::Rapns::Notification.update_all type: 'Rapns::Apns::Notification'
+    AddGcm::Rapns::App.update_all type: 'Rapns::Apns::App'
 
-    change_column :rapns_notifications, :type, :string, :null => false
-    change_column :rapns_apps, :type, :string, :null => false
-    change_column :rapns_notifications, :device_token, :string, { :null => true, :limit => 64 }
-    change_column :rapns_notifications, :expiry, :integer, { :null => true, :default => 1.day.to_i }
-    change_column :rapns_apps, :environment, :string, :null => true
-    change_column :rapns_apps, :certificate, :text, :null => true, :default => nil
+    change_column :rapns_notifications, :type, :string, null: false
+    change_column :rapns_apps, :type, :string, null: false
+    change_column :rapns_notifications, :device_token, :string, { null: true, limit: 64 }
+    change_column :rapns_notifications, :expiry, :integer, { null: true, default: 1.day.to_i }
+    change_column :rapns_apps, :environment, :string, null: true
+    change_column :rapns_apps, :certificate, :text, null: true, default: nil
 
-    change_column :rapns_notifications, :error_description, :text, :null => true, :default => nil
-    change_column :rapns_notifications, :sound, :string, :default => 'default'
+    change_column :rapns_notifications, :error_description, :text, null: true, default: nil
+    change_column :rapns_notifications, :sound, :string, default: 'default'
 
     rename_column :rapns_notifications, :attributes_for_device, :data
     rename_column :rapns_apps, :key, :name
 
-    add_column :rapns_apps, :auth_key, :string, :null => true
+    add_column :rapns_apps, :auth_key, :string, null: true
 
-    add_column :rapns_notifications, :collapse_key, :string, :null => true
-    add_column :rapns_notifications, :delay_while_idle, :boolean, :null => false, :default => false
+    add_column :rapns_notifications, :collapse_key, :string, null: true
+    add_column :rapns_notifications, :delay_while_idle, :boolean, null: false, default: false
 
     reg_ids_type = ActiveRecord::Base.connection.adapter_name.include?('Mysql') ? :mediumtext : :text
-    add_column :rapns_notifications, :registration_ids, reg_ids_type, :null => true
-    add_column :rapns_notifications, :app_id, :integer, :null => true
-    add_column :rapns_notifications, :retries, :integer, :null => true, :default => 0
+    add_column :rapns_notifications, :registration_ids, reg_ids_type, null: true
+    add_column :rapns_notifications, :app_id, :integer, null: true
+    add_column :rapns_notifications, :retries, :integer, null: true, default: 0
 
     AddGcm::Rapns::Notification.reset_column_information
     AddGcm::Rapns::App.reset_column_information
@@ -47,30 +47,30 @@ class AddGcm < ActiveRecord::Migration
       AddGcm::Rapns::Notification.update_all(['app_id = ?', app.id], ['app = ?', app.name])
     end
 
-    change_column :rapns_notifications, :app_id, :integer, :null => false
+    change_column :rapns_notifications, :app_id, :integer, null: false
     remove_column :rapns_notifications, :app
 
     if index_name_exists?(:rapns_notifications, "index_rapns_notifications_multi", true)
-      remove_index :rapns_notifications, :name => "index_rapns_notifications_multi"
+      remove_index :rapns_notifications, name: "index_rapns_notifications_multi"
     elsif index_name_exists?(:rapns_notifications, "index_rapns_notifications_on_delivered_failed_deliver_after", false)
-      remove_index :rapns_notifications, :name => "index_rapns_notifications_on_delivered_failed_deliver_after"
+      remove_index :rapns_notifications, name: "index_rapns_notifications_on_delivered_failed_deliver_after"
     end
-    add_index :rapns_notifications, [:app_id, :delivered, :failed, :deliver_after], :name => "index_rapns_notifications_multi"
+    add_index :rapns_notifications, [:app_id, :delivered, :failed, :deliver_after], name: "index_rapns_notifications_multi"
   end
 
   def self.down
-    AddGcm::Rapns::Notification.where(:type => 'Rapns::Gcm::Notification').delete_all
+    AddGcm::Rapns::Notification.where(type: 'Rapns::Gcm::Notification').delete_all
 
     remove_column :rapns_notifications, :type
     remove_column :rapns_apps, :type
 
-    change_column :rapns_notifications, :device_token, :string, { :null => false, :limit => 64 }
-    change_column :rapns_notifications, :expiry, :integer, { :null => false, :default => 1.day.to_i }
-    change_column :rapns_apps, :environment, :string, :null => false
-    change_column :rapns_apps, :certificate, :text, :null => false
+    change_column :rapns_notifications, :device_token, :string, { null: false, limit: 64 }
+    change_column :rapns_notifications, :expiry, :integer, { null: false, default: 1.day.to_i }
+    change_column :rapns_apps, :environment, :string, null: false
+    change_column :rapns_apps, :certificate, :text, null: false
 
-    change_column :rapns_notifications, :error_description, :string, :null => true, :default => nil
-    change_column :rapns_notifications, :sound, :string, :default => '1.aiff'
+    change_column :rapns_notifications, :error_description, :string, null: true, default: nil
+    change_column :rapns_notifications, :sound, :string, default: '1.aiff'
 
     rename_column :rapns_notifications, :data, :attributes_for_device
     rename_column :rapns_apps, :name, :key
@@ -82,7 +82,7 @@ class AddGcm < ActiveRecord::Migration
     remove_column :rapns_notifications, :registration_ids
     remove_column :rapns_notifications, :retries
 
-    add_column :rapns_notifications, :app, :string, :null => true
+    add_column :rapns_notifications, :app, :string, null: true
 
     AddGcm::Rapns::Notification.reset_column_information
     AddGcm::Rapns::App.reset_column_information
@@ -92,11 +92,11 @@ class AddGcm < ActiveRecord::Migration
     end
 
     if index_name_exists?(:rapns_notifications, :index_rapns_notifications_multi, true)
-        remove_index :rapns_notifications, :name => :index_rapns_notifications_multi
+      remove_index :rapns_notifications, name: :index_rapns_notifications_multi
     end
 
     remove_column :rapns_notifications, :app_id
 
-    add_index :rapns_notifications, [:delivered, :failed, :deliver_after], :name => :index_rapns_notifications_multi
+    add_index :rapns_notifications, [:delivered, :failed, :deliver_after], name: :index_rapns_notifications_multi
   end
 end

data/lib/generators/templates/add_rpush.rb

@@ -44,28 +44,28 @@ class AddRpush < ActiveRecord::Migration
   class CreateRapnsNotifications < ActiveRecord::Migration
     def self.up
       create_table :rapns_notifications do |t|
-        t.integer   :badge,                 :null => true
-        t.string    :device_token,          :null => false, :limit => 64
-        t.string    :sound,                 :null => true,  :default => "1.aiff"
-        t.string    :alert,                 :null => true
-        t.text      :attributes_for_device, :null => true
-        t.integer   :expiry,                :null => false, :default => 1.day.to_i
-        t.boolean   :delivered,             :null => false, :default => false
-        t.timestamp :delivered_at,          :null => true
-        t.boolean   :failed,                :null => false, :default => false
-        t.timestamp :failed_at,             :null => true
-        t.integer   :error_code,            :null => true
-        t.string    :error_description,     :null => true
-        t.timestamp :deliver_after,         :null => true
+        t.integer   :badge,                 null: true
+        t.string    :device_token,          null: false, limit: 64
+        t.string    :sound,                 null: true,  default: "1.aiff"
+        t.string    :alert,                 null: true
+        t.text      :attributes_for_device, null: true
+        t.integer   :expiry,                null: false, default: 1.day.to_i
+        t.boolean   :delivered,             null: false, default: false
+        t.timestamp :delivered_at,          null: true
+        t.boolean   :failed,                null: false, default: false
+        t.timestamp :failed_at,             null: true
+        t.integer   :error_code,            null: true
+        t.string    :error_description,     null: true
+        t.timestamp :deliver_after,         null: true
         t.timestamps
       end
 
-      add_index :rapns_notifications, [:delivered, :failed, :deliver_after], :name => 'index_rapns_notifications_multi'
+      add_index :rapns_notifications, [:delivered, :failed, :deliver_after], name: 'index_rapns_notifications_multi'
     end
 
     def self.down
       if index_name_exists?(:rapns_notifications, 'index_rapns_notifications_multi', true)
-        remove_index :rapns_notifications, :name => 'index_rapns_notifications_multi'
+        remove_index :rapns_notifications, name: 'index_rapns_notifications_multi'
       end
       drop_table :rapns_notifications
     end
@@ -74,8 +74,8 @@ class AddRpush < ActiveRecord::Migration
   class CreateRapnsFeedback < ActiveRecord::Migration
     def self.up
       create_table :rapns_feedback do |t|
-        t.string    :device_token,          :null => false, :limit => 64
-        t.timestamp :failed_at,             :null => false
+        t.string    :device_token,          null: false, limit: 64
+        t.timestamp :failed_at,             null: false
         t.timestamps
       end
 
@@ -92,7 +92,7 @@ class AddRpush < ActiveRecord::Migration
 
   class AddAlertIsJsonToRapnsNotifications < ActiveRecord::Migration
     def self.up
-      add_column :rapns_notifications, :alert_is_json, :boolean, :null => true, :default => false
+      add_column :rapns_notifications, :alert_is_json, :boolean, null: true, default: false
     end
 
     def self.down
@@ -102,8 +102,8 @@ class AddRpush < ActiveRecord::Migration
 
   class AddAppToRapns < ActiveRecord::Migration
     def self.up
-      add_column :rapns_notifications, :app, :string, :null => true
-      add_column :rapns_feedback, :app, :string, :null => true
+      add_column :rapns_notifications, :app, :string, null: true
+      add_column :rapns_feedback, :app, :string, null: true
     end
 
     def self.down
@@ -115,11 +115,11 @@ class AddRpush < ActiveRecord::Migration
   class CreateRapnsApps < ActiveRecord::Migration
     def self.up
       create_table :rapns_apps do |t|
-        t.string    :key,             :null => false
-        t.string    :environment,     :null => false
-        t.text      :certificate,     :null => false
-        t.string    :password,        :null => true
-        t.integer   :connections,     :null => false, :default => 1
+        t.string    :key,             null: false
+        t.string    :environment,     null: false
+        t.text      :certificate,     null: false
+        t.string    :password,        null: true
+        t.integer   :connections,     null: false, default: 1
         t.timestamps
       end
     end
@@ -142,66 +142,66 @@ class AddRpush < ActiveRecord::Migration
     end
 
     def self.up
-      add_column :rapns_notifications, :type, :string, :null => true
-      add_column :rapns_apps, :type, :string, :null => true
+      add_column :rapns_notifications, :type, :string, null: true
+      add_column :rapns_apps, :type, :string, null: true
 
-      AddGcm::Rapns::Notification.update_all :type => 'Rapns::Apns::Notification'
-      AddGcm::Rapns::App.update_all :type => 'Rapns::Apns::App'
+      AddGcm::Rapns::Notification.update_all type: 'Rapns::Apns::Notification'
+      AddGcm::Rapns::App.update_all type: 'Rapns::Apns::App'
 
-      change_column :rapns_notifications, :type, :string, :null => false
-      change_column :rapns_apps, :type, :string, :null => false
-      change_column :rapns_notifications, :device_token, :string, { :null => true, :limit => 64 }
-      change_column :rapns_notifications, :expiry, :integer, { :null => true, :default => 1.day.to_i }
-      change_column :rapns_apps, :environment, :string, :null => true
-      change_column :rapns_apps, :certificate, :text, :null => true, :default => nil
+      change_column :rapns_notifications, :type, :string, null: false
+      change_column :rapns_apps, :type, :string, null: false
+      change_column :rapns_notifications, :device_token, :string, { null: true, limit: 64 }
+      change_column :rapns_notifications, :expiry, :integer, { null: true, default: 1.day.to_i }
+      change_column :rapns_apps, :environment, :string, null: true
+      change_column :rapns_apps, :certificate, :text, null: true, default: nil
 
-      change_column :rapns_notifications, :error_description, :text, :null => true, :default => nil
-      change_column :rapns_notifications, :sound, :string, :default => 'default'
+      change_column :rapns_notifications, :error_description, :text, null: true, default: nil
+      change_column :rapns_notifications, :sound, :string, default: 'default'
 
       rename_column :rapns_notifications, :attributes_for_device, :data
       rename_column :rapns_apps, :key, :name
 
-      add_column :rapns_apps, :auth_key, :string, :null => true
+      add_column :rapns_apps, :auth_key, :string, null: true
 
-      add_column :rapns_notifications, :collapse_key, :string, :null => true
-      add_column :rapns_notifications, :delay_while_idle, :boolean, :null => false, :default => false
+      add_column :rapns_notifications, :collapse_key, :string, null: true
+      add_column :rapns_notifications, :delay_while_idle, :boolean, null: false, default: false
 
       reg_ids_type = ActiveRecord::Base.connection.adapter_name.include?('Mysql') ? :mediumtext : :text
-      add_column :rapns_notifications, :registration_ids, reg_ids_type, :null => true
-      add_column :rapns_notifications, :app_id, :integer, :null => true
-      add_column :rapns_notifications, :retries, :integer, :null => true, :default => 0
+      add_column :rapns_notifications, :registration_ids, reg_ids_type, null: true
+      add_column :rapns_notifications, :app_id, :integer, null: true
+      add_column :rapns_notifications, :retries, :integer, null: true, default: 0
 
       AddGcm::Rapns::Notification.reset_column_information
       AddGcm::Rapns::App.reset_column_information
 
       AddGcm::Rapns::App.all.each do |app|
-        AddGcm::Rapns::Notification.update_all(['app_id = ?', app.id], ['app = ?', app.name])
+        AddGcm::Rapns::Notification.where(app: app.name).update_all(app_id: app.id)
       end
 
-      change_column :rapns_notifications, :app_id, :integer, :null => false
+      change_column :rapns_notifications, :app_id, :integer, null: false
       remove_column :rapns_notifications, :app
 
       if index_name_exists?(:rapns_notifications, "index_rapns_notifications_multi", true)
-        remove_index :rapns_notifications, :name => "index_rapns_notifications_multi"
+        remove_index :rapns_notifications, name: "index_rapns_notifications_multi"
       elsif index_name_exists?(:rapns_notifications, "index_rapns_notifications_on_delivered_failed_deliver_after", false)
-        remove_index :rapns_notifications, :name => "index_rapns_notifications_on_delivered_failed_deliver_after"
+        remove_index :rapns_notifications, name: "index_rapns_notifications_on_delivered_failed_deliver_after"
       end
-      add_index :rapns_notifications, [:app_id, :delivered, :failed, :deliver_after], :name => "index_rapns_notifications_multi"
+      add_index :rapns_notifications, [:app_id, :delivered, :failed, :deliver_after], name: "index_rapns_notifications_multi"
     end
 
     def self.down
-      AddGcm::Rapns::Notification.where(:type => 'Rapns::Gcm::Notification').delete_all
+      AddGcm::Rapns::Notification.where(type: 'Rapns::Gcm::Notification').delete_all
 
       remove_column :rapns_notifications, :type
       remove_column :rapns_apps, :type
 
-      change_column :rapns_notifications, :device_token, :string, { :null => false, :limit => 64 }
-      change_column :rapns_notifications, :expiry, :integer, { :null => false, :default => 1.day.to_i }
-      change_column :rapns_apps, :environment, :string, :null => false
-      change_column :rapns_apps, :certificate, :text, :null => false
+      change_column :rapns_notifications, :device_token, :string, { null: false, limit: 64 }
+      change_column :rapns_notifications, :expiry, :integer, { null: false, default: 1.day.to_i }
+      change_column :rapns_apps, :environment, :string, null: false
+      change_column :rapns_apps, :certificate, :text, null: false
 
-      change_column :rapns_notifications, :error_description, :string, :null => true, :default => nil
-      change_column :rapns_notifications, :sound, :string, :default => '1.aiff'
+      change_column :rapns_notifications, :error_description, :string, null: true, default: nil
+      change_column :rapns_notifications, :sound, :string, default: '1.aiff'
 
       rename_column :rapns_notifications, :data, :attributes_for_device
       rename_column :rapns_apps, :name, :key
@@ -213,22 +213,22 @@ class AddRpush < ActiveRecord::Migration
       remove_column :rapns_notifications, :registration_ids
       remove_column :rapns_notifications, :retries
 
-      add_column :rapns_notifications, :app, :string, :null => true
+      add_column :rapns_notifications, :app, :string, null: true
 
       AddGcm::Rapns::Notification.reset_column_information
       AddGcm::Rapns::App.reset_column_information
 
       AddGcm::Rapns::App.all.each do |app|
-        AddGcm::Rapns::Notification.update_all(['app = ?', app.key], ['app_id = ?', app.id])
+        AddGcm::Rapns::Notification.where(app_id: app.id).update_all(app: app.key)
       end
 
       if index_name_exists?(:rapns_notifications, :index_rapns_notifications_multi, true)
-          remove_index :rapns_notifications, :name => :index_rapns_notifications_multi
+          remove_index :rapns_notifications, name: :index_rapns_notifications_multi
       end
 
       remove_column :rapns_notifications, :app_id
 
-      add_index :rapns_notifications, [:delivered, :failed, :deliver_after], :name => :index_rapns_notifications_multi
+      add_index :rapns_notifications, [:delivered, :failed, :deliver_after], name: :index_rapns_notifications_multi
     end
   end
 
@@ -240,11 +240,11 @@ class AddRpush < ActiveRecord::Migration
     end
 
     def self.up
-      add_column :rapns_notifications, :uri, :string, :null => true
+      add_column :rapns_notifications, :uri, :string, null: true
     end
 
     def self.down
-      AddWpns::Rapns::Notification.where(:type => 'Rapns::Wpns::Notification').delete_all
+      AddWpns::Rapns::Notification.where(type: 'Rapns::Wpns::Notification').delete_all
       remove_column :rapns_notifications, :uri
     end
   end
@@ -257,14 +257,14 @@ class AddRpush < ActiveRecord::Migration
     end
 
     def self.up
-      add_column :rapns_apps, :client_id, :string, :null => true
-      add_column :rapns_apps, :client_secret, :string, :null => true
-      add_column :rapns_apps, :access_token, :string, :null => true
-      add_column :rapns_apps, :access_token_expiration, :datetime, :null => true
+      add_column :rapns_apps, :client_id, :string, null: true
+      add_column :rapns_apps, :client_secret, :string, null: true
+      add_column :rapns_apps, :access_token, :string, null: true
+      add_column :rapns_apps, :access_token_expiration, :datetime, null: true
     end
 
     def self.down
-      AddAdm::Rapns::Notification.where(:type => 'Rapns::Adm::Notification').delete_all
+      AddAdm::Rapns::Notification.where(type: 'Rapns::Adm::Notification').delete_all
 
       remove_column :rapns_apps, :client_id
       remove_column :rapns_apps, :client_secret
@@ -339,7 +339,7 @@ class AddRpush < ActiveRecord::Migration
 
   class AddFailAfterToRpushNotifications < ActiveRecord::Migration
     def self.up
-      add_column :rpush_notifications, :fail_after, :timestamp, :null => true
+      add_column :rpush_notifications, :fail_after, :timestamp, null: true
     end
 
     def self.down