aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMatthew Draper <matthew@trebex.net>2016-01-22 04:14:03 +1030
committerMatthew Draper <matthew@trebex.net>2016-01-24 22:54:04 +1030
commit503fe757c7f5f917deab95acdcd421a1dede05c7 (patch)
tree40e3ee39aecebd0967a932e1516271c5d44c134d
parent322dca293b3716ccaa09e7e82046e539b0d2ffda (diff)
downloadrails-503fe757c7f5f917deab95acdcd421a1dede05c7.tar.gz
rails-503fe757c7f5f917deab95acdcd421a1dede05c7.tar.bz2
rails-503fe757c7f5f917deab95acdcd421a1dede05c7.zip
Ditch the EM error logging helper
We're no longer doing our work in the EM event loop, so errors are quite unlikely, and if they do occur, they're not really our responsibility to handle.
-rw-r--r--actioncable/lib/action_cable/process/logging.rb8
-rw-r--r--actioncable/test/subscription_adapter/common.rb1
-rw-r--r--railties/lib/rails/generators/rails/app/templates/config.ru.tt3
3 files changed, 1 insertions, 11 deletions
diff --git a/actioncable/lib/action_cable/process/logging.rb b/actioncable/lib/action_cable/process/logging.rb
deleted file mode 100644
index ebb8990891..0000000000
--- a/actioncable/lib/action_cable/process/logging.rb
+++ /dev/null
@@ -1,8 +0,0 @@
-require 'action_cable/server'
-
-if defined?(::EventMachine)
- EventMachine.error_handler do |e|
- puts "Error raised inside the event loop: #{e.message}"
- puts e.backtrace.join("\n")
- end
-end
diff --git a/actioncable/test/subscription_adapter/common.rb b/actioncable/test/subscription_adapter/common.rb
index 5efc6d6b81..361858784e 100644
--- a/actioncable/test/subscription_adapter/common.rb
+++ b/actioncable/test/subscription_adapter/common.rb
@@ -1,7 +1,6 @@
require 'test_helper'
require 'concurrent'
-require 'action_cable/process/logging'
require 'active_support/core_ext/hash/indifferent_access'
require 'pathname'
diff --git a/railties/lib/rails/generators/rails/app/templates/config.ru.tt b/railties/lib/rails/generators/rails/app/templates/config.ru.tt
index 70556fcc99..343c0833d7 100644
--- a/railties/lib/rails/generators/rails/app/templates/config.ru.tt
+++ b/railties/lib/rails/generators/rails/app/templates/config.ru.tt
@@ -3,9 +3,8 @@
require ::File.expand_path('../config/environment', __FILE__)
<%- unless options[:skip_action_cable] -%>
-# Action Cable uses EventMachine which requires that all classes are loaded in advance
+# Action Cable requires that all classes are loaded in advance
Rails.application.eager_load!
-require 'action_cable/process/logging'
<%- end -%>
run Rails.application