use ruby 2.4

.ruby-version

@@ -1 +1 @@
-2.3.3
+2.4.1

Gemfile.lock

@@ -613,7 +613,7 @@ DEPENDENCIES
   webmock
 
 RUBY VERSION
-   ruby 2.3.3p222
+   ruby 2.4.1p111
 
 BUNDLED WITH
    1.14.6

app/models/concerns/attr_encrypted_support.rb

@@ -2,7 +2,7 @@
 require 'attr_encrypted'
 
 module AttrEncryptedSupport
-  ENCRYPTION_KEY = Rails.application.secrets.secret_key_base
+  ENCRYPTION_KEY = Rails.application.secrets.secret_key_base[0...32]
   ENCRYPTION_KEY_SHA = Digest::SHA2.hexdigest(Rails.application.secrets.secret_key_base)
 
   def self.included(base)

app/models/user.rb

@@ -34,9 +34,12 @@ def object_attrs_for_paper_trail
   validates :external_id, presence: :true, unless: :integration?
 
   scope :search, ->(query) {
-    return self if query.blank?
-    query = ActiveRecord::Base.send(:sanitize_sql_like, query)
-    where("name LIKE ? OR email LIKE ?", "%#{query}%", "%#{query}%")
+    if query.blank?
+      self
+    else
+      query = ActiveRecord::Base.send(:sanitize_sql_like, query)
+      where("name LIKE ? OR email LIKE ?", "%#{query}%", "%#{query}%")
+    end
   }
 
   def self.with_role(role_id, project_id)

plugins/flowdock/test/samson_flowdock/flowdock_service_test.rb

@@ -21,9 +21,7 @@
 
     it "shows nothing when flowdock fails to fetch users" do
       stub_request(:get, "https://api.flowdock.com/v1/users").to_timeout
-      Rails.logger.expects(:error).with(
-        'Error fetching flowdock users (token invalid / flowdock down). Net::OpenTimeout: execution expired'
-      )
+      Rails.logger.expects(:error).with(includes('Error fetching flowdock users'))
       service.users.must_equal([])
 
       # is cached

plugins/slack_webhooks/test/samson_slack_webhooks/slack_webhooks_service_test.rb

@@ -26,7 +26,7 @@
     it "returns empty users when there's an error" do
       stub_request(:post, "https://slack.com/api/users.list").
         to_return(body: {ok: false, error: "some error"}.to_json)
-      Rails.logger.expects(:error).with('Error fetching slack users: some error')
+      Rails.logger.expects(:error).with(includes('Error fetching slack users'))
       service.users.must_equal([])
 
       # is cached
@@ -35,9 +35,7 @@
 
     it "shows nothing when slack fails to fetch users" do
       stub_request(:post, "https://slack.com/api/users.list").to_timeout
-      Rails.logger.expects(:error).with(
-        'Error fetching slack users (token invalid / service down). Faraday::ConnectionFailed: execution expired'
-      )
+      Rails.logger.expects(:error).with(includes('Error fetching slack users'))
       service.users.must_equal([])
 
       # is cached