rpush 4.2.0 → 7.0.1

data/README.md

@@ -1,5 +1,5 @@
 [![Gem Version](https://badge.fury.io/rb/rpush.svg)](http://badge.fury.io/rb/rpush)
-[![Build Status](https://travis-ci.org/rpush/rpush.svg?branch=master)](https://travis-ci.org/rpush/rpush)
+[![RPush Test](https://github.com/rpush/rpush/actions/workflows/test.yml/badge.svg)](https://github.com/rpush/rpush/actions/workflows/test.yml)
 [![Test Coverage](https://codeclimate.com/github/rpush/rpush/badges/coverage.svg)](https://codeclimate.com/github/rpush/rpush)
 [![Code Climate](https://codeclimate.com/github/rpush/rpush/badges/gpa.svg)](https://codeclimate.com/github/rpush/rpush)
 [![Join the chat at https://gitter.im/rpush/rpush](https://badges.gitter.im/Join%20Chat.svg)](https://gitter.im/rpush/rpush?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge)
@@ -18,12 +18,13 @@ Rpush aims to be the *de facto* gem for sending push notifications in Ruby. Its
   * [**Amazon Device Messaging**](#amazon-device-messaging)
   * [**Windows Phone Push Notification Service**](#windows-phone-notification-service)
   * [**Pushy**](#pushy)
+  * [**Webpush**](#webpush)
 
 #### Feature Highlights
 
 * Use [**ActiveRecord**](https://github.com/rpush/rpush/wiki/Using-ActiveRecord) or [**Redis**](https://github.com/rpush/rpush/wiki/Using-Redis) for storage.
 * Plugins for [**Bugsnag**](https://github.com/rpush/rpush-plugin-bugsnag),
-[**Sentry**](https://github.com/rpush/rpush-plugin-sentry), [**StatsD**](https://github.com/rpush/rpush-plugin-statsd) or [write your own](https://github.com/rpush/rpush/wiki/Writing-a-Plugin).
+[**Sentry**](https://github.com/rpush/rpush-plugin-sentry), [**StatsD**](https://github.com/rpush/rpush-plugin-statsd). Third party plugins: [**Prometheus Exporter**](https://github.com/equinux/rpush-plugin-prometheus-exporter). Or [write your own](https://github.com/rpush/rpush/wiki/Writing-a-Plugin).
 * Seamless integration with your projects, including **Rails**.
 * Run as a [daemon](https://github.com/rpush/rpush#as-a-daemon), inside a [job queue](https://github.com/rpush/rpush/wiki/Push-API), on the [command-line](https://github.com/rpush/rpush#on-the-command-line) or [embedded](https://github.com/rpush/rpush/wiki/Embedding-API) in another process.
 * Scales vertically (threading) and horizontally (multiple processes).
@@ -52,56 +53,98 @@ $ bundle exec rpush init
 
 #### Apple Push Notification Service
 
+There is a choice of two modes (and one legacy mode) using certificates or using tokens:
 
-If this is your first time using the APNs, you will need to generate SSL certificates. See [Generating Certificates](https://github.com/rpush/rpush/wiki/Generating-Certificates) for instructions.
+* `Rpush::Apns2` This requires an annually renewable certificate. see https://developer.apple.com/documentation/usernotifications/setting_up_a_remote_notification_server/establishing_a_certificate-based_connection_to_apns
+* `Rpush::Apnsp8` This uses encrypted tokens and requires an encryption key id and encryption key (provide as a p8 file). (see https://developer.apple.com/documentation/usernotifications/setting_up_a_remote_notification_server/establishing_a_token-based_connection_to_apns)
+* `Rpush::Apns` There is also the original APNS (the original version using certificates with a binary underlying protocol over TCP directly rather than over Http/2).
+  Apple have [announced](https://developer.apple.com/news/?id=c88acm2b) that this is not supported after March 31, 2021.
+
+If this is your first time using the APNs, you will need to generate either SSL certificates (for Apns2 or Apns) or an Encryption Key (p8) and an Encryption Key ID (for Apnsp8). See [Generating Certificates](https://github.com/rpush/rpush/wiki/Generating-Certificates) for instructions.
+
+##### Apnsp8
+
+To use the p8 APNs Api:
 
 ```ruby
-app = Rpush::Apns::App.new
+app = Rpush::Apnsp8::App.new
 app.name = "ios_app"
-app.certificate = File.read("/path/to/sandbox.pem")
+app.apn_key = File.read("/path/to/sandbox.p8")
 app.environment = "development" # APNs environment.
-app.password = "certificate password"
+app.apn_key_id = "APN KEY ID" # This is the Encryption Key ID provided by apple
+app.team_id = "TEAM ID" # the team id - e.g. ABCDE12345
+app.bundle_id = "BUNDLE ID" # the unique bundle id of the app, like com.example.appname
 app.connections = 1
 app.save!
 ```
 
 ```ruby
-n = Rpush::Apns::Notification.new
-n.app = Rpush::Apns::App.find_by_name("ios_app")
+n = Rpush::Apnsp8::Notification.new
+n.app = Rpush::Apnsp8::App.find_by_name("ios_app")
 n.device_token = "..." # hex string
 n.alert = "hi mom!"
 n.data = { foo: :bar }
 n.save!
 ```
 
-The `url_args` attribute is available for Safari Push Notifications.
+##### Apns2
 
-You should also implement the [ssl_certificate_will_expire](https://github.com/rpush/rpush/wiki/Reflection-API) reflection to monitor when your certificate is due to expire.
+(NB this uses the same protocol as Apnsp8, but authenticates with a certificate rather than tokens)
+
+```ruby
+app = Rpush::Apns2::App.new
+app.name = "ios_app"
+app.certificate = File.read("/path/to/sandbox.pem")
+app.environment = "development"
+app.password = "certificate password"
+app.bundle_id = "BUNDLE ID" # the unique bundle id of the app, like com.example.appname
+app.connections = 1
+app.save!
+```
+
+```ruby
+n = Rpush::Apns2::Notification.new
+n.app = Rpush::Apns2::App.find_by_name("ios_app")
+n.device_token = "..." # hex string
+n.alert = "hi mom!"
+n.data = {
+  headers: { 'apns-topic': "BUNDLE ID" }, # the bundle id of the app, like com.example.appname. Not necessary if set on the app (see above)
+  foo: :bar
+}
+n.save!
+```
 
-To use the newer APNs Api replace `Rpush::Apns::App` with `Rpush::Apns2::App`.
+You should also implement the [ssl_certificate_will_expire](https://github.com/rpush/rpush/wiki/Reflection-API) reflection to monitor when your certificate is due to expire.
 
-To use the p8 APNs Api replace `Rpush::Apns::App` with `Rpush::Apnsp8::App`.
+##### Apns (legacy protocol)
 
 ```ruby
-app = Rpush::Apnsp8::App.new
+app = Rpush::Apns::App.new
 app.name = "ios_app"
-app.apn_key = File.read("/path/to/sandbox.p8")
+app.certificate = File.read("/path/to/sandbox.pem")
 app.environment = "development" # APNs environment.
-app.apn_key_id = "APN KEY ID"
-app.team_id = "TEAM ID"
-app.bundle_id = "BUNDLE ID"
+app.password = "certificate password"
 app.connections = 1
 app.save!
 ```
 
 ```ruby
 n = Rpush::Apns::Notification.new
-n.app = Rpush::Apnsp8::App.find_by_name("ios_app")
+n.app = Rpush::Apns::App.find_by_name("ios_app")
 n.device_token = "..." # hex string
 n.alert = "hi mom!"
 n.data = { foo: :bar }
 n.save!
 ```
+
+##### Safari Push Notifications
+
+Using one of the notifications methods above, the `url_args` attribute is available for Safari Push Notifications.
+
+##### Environment
+
+The app `environment` for any Apns* option is "development" for XCode installs, and "production" for app store and TestFlight. Note that for Apns2 you can now use one (production + sandbox) certificate (you don't need a separate "sandbox" or development certificate), but if you do generate a development/sandbox certificate it can only be used for "development". With Apnsp8 tokens, you can target either "development" or "production" environments.
+
 #### Firebase Cloud Messaging
 
 FCM and GCM are – as of writing – compatible with each other. See also [this comment](https://github.com/rpush/rpush/issues/284#issuecomment-228330206) for further references.
@@ -208,7 +251,7 @@ n.save!
 
 #### Windows Raw Push Notifications
 
-Note: The data is passed as `.to_json` so only this format is supported, altough raw notifications are meant to support any kind of data.
+Note: The data is passed as `.to_json` so only this format is supported, although raw notifications are meant to support any kind of data.
 Current data structure enforces hashes and `.to_json` representation is natural presentation of it.
 
 ```ruby
@@ -256,6 +299,49 @@ n.save!
 
 For more documentation on [Pushy](https://pushy.me/docs).
 
+#### Webpush
+
+[Webpush](https://tools.ietf.org/html/draft-ietf-webpush-protocol-10) is a
+protocol for delivering push messages to desktop browsers. It's supported by
+all major browsers (except Safari, you have to use one of the Apns transports
+for that).
+
+Using [VAPID](https://tools.ietf.org/html/draft-ietf-webpush-vapid-01), there
+is no need for the sender of push notifications to register upfront with push
+services (as was the case with the now legacy Mozilla or Google desktop push
+providers).
+
+Instead, you generate a pair of keys and use the public key when subscribing
+users in your web app. The keys are stored along with an email address (which,
+according to the spec, can be used by push service providers to contact you in
+case of problems) in the `certificates` field of the Rpush Application record:
+
+```ruby
+vapid_keypair = Webpush.generate_key.to_hash
+app = Rpush::Webpush::App.new
+app.name = 'webpush'
+app.certificate = vapid_keypair.merge(subject: 'user@example.org').to_json
+app.connections = 1
+app.save!
+```
+
+The `subscription` object you obtain from a subscribed browser holds an
+endpoint URL and cryptographic keys. When sending a notification, simply pass
+the whole subscription as sole member of the `registration_ids` collection:
+
+```ruby
+n = Rpush::Webpush::Notification.new
+n.app = Rpush::App.find_by_name("webpush")
+n.registration_ids = [subscription]
+n.data = { message: "hi mom!" }
+n.save!
+```
+
+In order to send the same message to multiple devices, create one
+`Notification` per device, as passing multiple subscriptions at once as
+`registration_ids` is not supported.
+
+
 ### 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.

data/lib/generators/templates/add_adm.rb

@@ -1,4 +1,4 @@
-class AddAdm < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0] : ActiveRecord::Migration
+class AddAdm < ActiveRecord::Migration[5.0]
   module Rapns
     class Notification < ActiveRecord::Base
       self.table_name = 'rapns_notifications'

data/lib/generators/templates/add_alert_is_json_to_rapns_notifications.rb

@@ -1,4 +1,4 @@
-class AddAlertIsJsonToRapnsNotifications < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0] : ActiveRecord::Migration
+class AddAlertIsJsonToRapnsNotifications < ActiveRecord::Migration[5.0]
   def self.up
     add_column :rapns_notifications, :alert_is_json, :boolean, null: true, default: false
   end
@@ -6,4 +6,4 @@ class AddAlertIsJsonToRapnsNotifications < ActiveRecord::VERSION::MAJOR >= 5 ? A
   def self.down
     remove_column :rapns_notifications, :alert_is_json
   end
-end
+end

data/lib/generators/templates/add_app_to_rapns.rb

@@ -1,4 +1,4 @@
-class AddAppToRapns < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0] : ActiveRecord::Migration
+class AddAppToRapns < ActiveRecord::Migration[5.0]
   def self.up
     add_column :rapns_notifications, :app, :string, null: true
     add_column :rapns_feedback, :app, :string, null: true
@@ -8,4 +8,4 @@ class AddAppToRapns < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migratio
     remove_column :rapns_notifications, :app
     remove_column :rapns_feedback, :app
   end
-end
+end

data/lib/generators/templates/add_fail_after_to_rpush_notifications.rb

@@ -1,4 +1,4 @@
-class AddFailAfterToRpushNotifications < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0] : ActiveRecord::Migration
+class AddFailAfterToRpushNotifications < ActiveRecord::Migration[5.0]
   def self.up
     add_column :rpush_notifications, :fail_after, :timestamp, null: true
   end

data/lib/generators/templates/add_gcm.rb

@@ -1,4 +1,4 @@
-class AddGcm < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0] : ActiveRecord::Migration
+class AddGcm < ActiveRecord::Migration[5.0]
   module Rapns
     class App < ActiveRecord::Base
       self.table_name = 'rapns_apps'
@@ -19,8 +19,8 @@ class AddGcm < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0]
 
     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_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
 
@@ -50,18 +50,10 @@ class AddGcm < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0]
     change_column :rapns_notifications, :app_id, :integer, null: false
     remove_column :rapns_notifications, :app
 
-    if ActiveRecord.version >= Gem::Version.new('5.1')
-      if index_name_exists?(:rapns_notifications, "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")
-        remove_index :rapns_notifications, name: "index_rapns_notifications_on_delivered_failed_deliver_after"
-      end
-    else
-      if index_name_exists?(:rapns_notifications, "index_rapns_notifications_multi", true)
-        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"
-      end
+    if index_name_exists?(:rapns_notifications, "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")
+      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"
@@ -73,8 +65,8 @@ class AddGcm < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0]
     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_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
 
@@ -100,14 +92,8 @@ class AddGcm < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0]
       AddGcm::Rapns::Notification.update_all(['app = ?', app.key], ['app_id = ?', app.id])
     end
 
-    if ActiveRecord.version >= Gem::Version.new('5.1')
-      if index_name_exists?(:rapns_notifications, :index_rapns_notifications_multi)
-        remove_index :rapns_notifications, name: :index_rapns_notifications_multi
-      end
-    else
-      if index_name_exists?(:rapns_notifications, :index_rapns_notifications_multi, true)
-        remove_index :rapns_notifications, name: :index_rapns_notifications_multi
-      end
+    if index_name_exists?(:rapns_notifications, :index_rapns_notifications_multi)
+      remove_index :rapns_notifications, name: :index_rapns_notifications_multi
     end
 
     remove_column :rapns_notifications, :app_id

data/lib/generators/templates/add_rpush.rb

@@ -19,7 +19,7 @@
 # approach. The constituent parts of this migration have been executed
 # many times, by many people!
 
-class AddRpush < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0] : ActiveRecord::Migration
+class AddRpush < ActiveRecord::Migration[5.0]
   def self.migrations
     [CreateRapnsNotifications, CreateRapnsFeedback,
      AddAlertIsJsonToRapnsNotifications, AddAppToRapns,
@@ -41,7 +41,7 @@ class AddRpush < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0
     end
   end
 
-  class CreateRapnsNotifications < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0] : ActiveRecord::Migration
+  class CreateRapnsNotifications < ActiveRecord::Migration[5.0]
     def self.up
       create_table :rapns_notifications do |t|
         t.integer   :badge,                 null: true
@@ -64,21 +64,15 @@ class AddRpush < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0
     end
 
     def self.down
-      if ActiveRecord.version >= Gem::Version.new('5.1')
-        if index_name_exists?(:rapns_notifications, 'index_rapns_notifications_multi')
-          remove_index :rapns_notifications, name: 'index_rapns_notifications_multi'
-        end
-      else
-        if index_name_exists?(:rapns_notifications, 'index_rapns_notifications_multi', true)
-          remove_index :rapns_notifications, name: 'index_rapns_notifications_multi'
-        end
+      if index_name_exists?(:rapns_notifications, 'index_rapns_notifications_multi')
+        remove_index :rapns_notifications, name: 'index_rapns_notifications_multi'
       end
 
       drop_table :rapns_notifications
     end
   end
 
-  class CreateRapnsFeedback < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0] : ActiveRecord::Migration
+  class CreateRapnsFeedback < ActiveRecord::Migration[5.0]
     def self.up
       create_table :rapns_feedback do |t|
         t.string    :device_token,          null: false, limit: 64
@@ -90,21 +84,15 @@ class AddRpush < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0
     end
 
     def self.down
-      if ActiveRecord.version >= Gem::Version.new('5.1')
-        if index_name_exists?(:rapns_feedback, :index_rapns_feedback_on_device_token)
-          remove_index :rapns_feedback, name: :index_rapns_feedback_on_device_token
-        end
-      else
-        if index_name_exists?(:rapns_feedback, :index_rapns_feedback_on_device_token, true)
-          remove_index :rapns_feedback, name: :index_rapns_feedback_on_device_token
-        end
+      if index_name_exists?(:rapns_feedback, :index_rapns_feedback_on_device_token)
+        remove_index :rapns_feedback, name: :index_rapns_feedback_on_device_token
       end
 
       drop_table :rapns_feedback
     end
   end
 
-  class AddAlertIsJsonToRapnsNotifications < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0] : ActiveRecord::Migration
+  class AddAlertIsJsonToRapnsNotifications < ActiveRecord::Migration[5.0]
     def self.up
       add_column :rapns_notifications, :alert_is_json, :boolean, null: true, default: false
     end
@@ -114,7 +102,7 @@ class AddRpush < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0
     end
   end
 
-  class AddAppToRapns < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0] : ActiveRecord::Migration
+  class AddAppToRapns < ActiveRecord::Migration[5.0]
     def self.up
       add_column :rapns_notifications, :app, :string, null: true
       add_column :rapns_feedback, :app, :string, null: true
@@ -126,7 +114,7 @@ class AddRpush < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0
     end
   end
 
-  class CreateRapnsApps < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0] : ActiveRecord::Migration
+  class CreateRapnsApps < ActiveRecord::Migration[5.0]
     def self.up
       create_table :rapns_apps do |t|
         t.string    :key,             null: false
@@ -143,7 +131,7 @@ class AddRpush < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0
     end
   end
 
-  class AddGcm < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0] : ActiveRecord::Migration
+  class AddGcm < ActiveRecord::Migration[5.0]
     module Rapns
       class App < ActiveRecord::Base
         self.table_name = 'rapns_apps'
@@ -164,8 +152,8 @@ class AddRpush < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0
 
       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_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
 
@@ -195,18 +183,10 @@ class AddRpush < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0
       change_column :rapns_notifications, :app_id, :integer, null: false
       remove_column :rapns_notifications, :app
 
-      if ActiveRecord.version >= Gem::Version.new('5.1')
-        if index_name_exists?(:rapns_notifications, "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")
-          remove_index :rapns_notifications, name: "index_rapns_notifications_on_delivered_failed_deliver_after"
-        end
-      else
-        if index_name_exists?(:rapns_notifications, "index_rapns_notifications_multi", true)
-          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"
-        end
+      if index_name_exists?(:rapns_notifications, "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")
+        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"
@@ -218,8 +198,8 @@ class AddRpush < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0
       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_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
 
@@ -245,14 +225,8 @@ class AddRpush < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0
         AddGcm::Rapns::Notification.where(app_id: app.id).update_all(app: app.key)
       end
 
-      if ActiveRecord.version >= Gem::Version.new('5.1')
-        if index_name_exists?(:rapns_notifications, :index_rapns_notifications_multi)
-          remove_index :rapns_notifications, name: :index_rapns_notifications_multi
-        end
-      else
-        if index_name_exists?(:rapns_notifications, :index_rapns_notifications_multi, true)
-          remove_index :rapns_notifications, name: :index_rapns_notifications_multi
-        end
+      if index_name_exists?(:rapns_notifications, :index_rapns_notifications_multi)
+        remove_index :rapns_notifications, name: :index_rapns_notifications_multi
       end
 
       remove_column :rapns_notifications, :app_id
@@ -261,7 +235,7 @@ class AddRpush < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0
     end
   end
 
-  class AddWpns < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0] : ActiveRecord::Migration
+  class AddWpns < ActiveRecord::Migration[5.0]
     module Rapns
       class Notification < ActiveRecord::Base
         self.table_name = 'rapns_notifications'
@@ -278,7 +252,7 @@ class AddRpush < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0
     end
   end
 
-  class AddAdm < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0] : ActiveRecord::Migration
+  class AddAdm < ActiveRecord::Migration[5.0]
     module Rapns
       class Notification < ActiveRecord::Base
         self.table_name = 'rapns_notifications'
@@ -302,7 +276,7 @@ class AddRpush < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0
     end
   end
 
-  class RenameRapnsToRpush < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0] : ActiveRecord::Migration
+  class RenameRapnsToRpush < ActiveRecord::Migration[5.0]
     module Rpush
       class App < ActiveRecord::Base
         self.table_name = 'rpush_apps'
@@ -322,24 +296,12 @@ class AddRpush < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0
       rename_table :rapns_apps, :rpush_apps
       rename_table :rapns_feedback, :rpush_feedback
 
-      if ActiveRecord.version >= Gem::Version.new('5.1')
-        if index_name_exists?(:rpush_notifications, :index_rapns_notifications_multi)
-          rename_index :rpush_notifications, :index_rapns_notifications_multi, :index_rpush_notifications_multi
-        end
-      else
-        if index_name_exists?(:rpush_notifications, :index_rapns_notifications_multi, true)
-          rename_index :rpush_notifications, :index_rapns_notifications_multi, :index_rpush_notifications_multi
-        end
+      if index_name_exists?(:rpush_notifications, :index_rapns_notifications_multi)
+        rename_index :rpush_notifications, :index_rapns_notifications_multi, :index_rpush_notifications_multi
       end
 
-      if ActiveRecord.version >= Gem::Version.new('5.1')
-        if index_name_exists?(:rpush_feedback, :index_rapns_feedback_on_device_token)
-          rename_index :rpush_feedback, :index_rapns_feedback_on_device_token, :index_rpush_feedback_on_device_token
-        end
-      else
-        if index_name_exists?(:rpush_feedback, :index_rapns_feedback_on_device_token, true)
-          rename_index :rpush_feedback, :index_rapns_feedback_on_device_token, :index_rpush_feedback_on_device_token
-        end
+      if index_name_exists?(:rpush_feedback, :index_rapns_feedback_on_device_token)
+        rename_index :rpush_feedback, :index_rapns_feedback_on_device_token, :index_rpush_feedback_on_device_token
       end
 
       update_type(RenameRapnsToRpush::Rpush::Notification, 'Rapns::Apns::Notification', 'Rpush::Apns::Notification')
@@ -364,24 +326,12 @@ class AddRpush < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0
       update_type(RenameRapnsToRpush::Rpush::App, 'Rpush::Adm::App', 'Rapns::Adm::App')
       update_type(RenameRapnsToRpush::Rpush::App, 'Rpush::Wpns::App', 'Rapns::Wpns::App')
 
-      if ActiveRecord.version >= Gem::Version.new('5.1')
-        if index_name_exists?(:rpush_notifications, :index_rpush_notifications_multi)
-          rename_index :rpush_notifications, :index_rpush_notifications_multi, :index_rapns_notifications_multi
-        end
-      else
-        if index_name_exists?(:rpush_notifications, :index_rpush_notifications_multi, true)
-          rename_index :rpush_notifications, :index_rpush_notifications_multi, :index_rapns_notifications_multi
-        end
+      if index_name_exists?(:rpush_notifications, :index_rpush_notifications_multi)
+        rename_index :rpush_notifications, :index_rpush_notifications_multi, :index_rapns_notifications_multi
       end
 
-      if ActiveRecord.version >= Gem::Version.new('5.1')
-        if index_name_exists?(:rpush_feedback, :index_rpush_feedback_on_device_token)
-          rename_index :rpush_feedback, :index_rpush_feedback_on_device_token, :index_rapns_feedback_on_device_token
-        end
-      else
-        if index_name_exists?(:rpush_feedback, :index_rpush_feedback_on_device_token, true)
-          rename_index :rpush_feedback, :index_rpush_feedback_on_device_token, :index_rapns_feedback_on_device_token
-        end
+      if index_name_exists?(:rpush_feedback, :index_rpush_feedback_on_device_token)
+        rename_index :rpush_feedback, :index_rpush_feedback_on_device_token, :index_rapns_feedback_on_device_token
       end
 
       rename_table :rpush_notifications, :rapns_notifications
@@ -390,7 +340,7 @@ class AddRpush < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0
     end
   end
 
-  class AddFailAfterToRpushNotifications < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0] : ActiveRecord::Migration
+  class AddFailAfterToRpushNotifications < ActiveRecord::Migration[5.0]
     def self.up
       add_column :rpush_notifications, :fail_after, :timestamp, null: true
     end

data/lib/generators/templates/add_wpns.rb

@@ -1,4 +1,4 @@
-class AddWpns < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0] : ActiveRecord::Migration
+class AddWpns < ActiveRecord::Migration[5.0]
   module Rapns
     class Notification < ActiveRecord::Base
       self.table_name = 'rapns_notifications'

data/lib/generators/templates/create_rapns_apps.rb

@@ -1,4 +1,4 @@
-class CreateRapnsApps < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0] : ActiveRecord::Migration
+class CreateRapnsApps < ActiveRecord::Migration[5.0]
   def self.up
     create_table :rapns_apps do |t|
       t.string    :key,             null: false

data/lib/generators/templates/create_rapns_feedback.rb

@@ -1,4 +1,4 @@
-class CreateRapnsFeedback < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0] : ActiveRecord::Migration
+class CreateRapnsFeedback < ActiveRecord::Migration[5.0]
   def self.up
     create_table :rapns_feedback do |t|
       t.string    :device_token,          null: false, limit: 64
@@ -10,14 +10,8 @@ class CreateRapnsFeedback < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Mi
   end
 
   def self.down
-    if ActiveRecord.version >= Gem::Version.new('5.1')
-      if index_name_exists?(:rapns_feedback, :index_rapns_feedback_on_device_token)
-        remove_index :rapns_feedback, name: :index_rapns_feedback_on_device_token
-      end
-    else
-      if index_name_exists?(:rapns_feedback, :index_rapns_feedback_on_device_token, true)
-        remove_index :rapns_feedback, name: :index_rapns_feedback_on_device_token
-      end
+    if index_name_exists?(:rapns_feedback, :index_rapns_feedback_on_device_token)
+      remove_index :rapns_feedback, name: :index_rapns_feedback_on_device_token
     end
 
     drop_table :rapns_feedback

data/lib/generators/templates/create_rapns_notifications.rb

@@ -1,4 +1,4 @@
-class CreateRapnsNotifications < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecord::Migration[5.0] : ActiveRecord::Migration
+class CreateRapnsNotifications < ActiveRecord::Migration[5.0]
   def self.up
     create_table :rapns_notifications do |t|
       t.integer   :badge,                 null: true
@@ -21,14 +21,8 @@ class CreateRapnsNotifications < ActiveRecord::VERSION::MAJOR >= 5 ? ActiveRecor
   end
 
   def self.down
-    if ActiveRecord.version >= Gem::Version.new('5.1')
-      if index_name_exists?(:rapns_notifications, 'index_rapns_notifications_multi')
-        remove_index :rapns_notifications, name: 'index_rapns_notifications_multi'
-      end
-    else
-      if index_name_exists?(:rapns_notifications, 'index_rapns_notifications_multi', true)
-        remove_index :rapns_notifications, name: 'index_rapns_notifications_multi'
-      end
+    if index_name_exists?(:rapns_notifications, 'index_rapns_notifications_multi')
+      remove_index :rapns_notifications, name: 'index_rapns_notifications_multi'
     end
 
     drop_table :rapns_notifications