Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Handle more IO errors (especially, ECONNRESET) | Matthew Draper | 2016-01-30 | 1 | -21/+47 |
| | | | | | | Also, address the possibility of the listen thread dying and needing to be respawned. As a bonus, we now defer construction of the thread until we are first given something to monitor. | ||||
* | Synchronize the lazy setters in Server | Matthew Draper | 2016-01-30 | 5 | -15/+47 |
| | | | | They're all at risk of races on the first requests. | ||||
* | Fix arguments to on_close | Matthew Draper | 2016-01-30 | 2 | -5/+3 |
| | |||||
* | Revert "Revert "Eliminate the EventMachine dependency"" | Matthew Draper | 2016-01-30 | 16 | -46/+349 |
| | |||||
* | Revert "Eliminate the EventMachine dependency" | David Heinemeier Hansson | 2016-01-27 | 16 | -349/+46 |
| | |||||
* | Merge branch '5-0-beta-sec' | Aaron Patterson | 2016-01-25 | 1 | -1/+1 |
|\ | | | | | | | | | | | | | | | | | | | | | * 5-0-beta-sec: bumping version fix version update task to deal with .beta1.1 Eliminate instance level writers for class accessors allow :file to be outside rails root, but anything else must be inside the rails view directory Don't short-circuit reject_if proc stop caching mime types globally use secure string comparisons for basic auth username / password | ||||
| * | bumping version | Aaron Patterson | 2016-01-25 | 1 | -1/+1 |
| | | |||||
* | | Merge pull request #23185 from droptheplot/actioncable-custom-channels | Kasper Timm Hansen | 2016-01-24 | 1 | -3/+5 |
|\ \ | | | | | | | ActionCable: Allow adding custom paths for channels | ||||
| * | | Rename channels_path var and fix channel_paths method for ActionCable config | Sergey Novikov | 2016-01-23 | 1 | -5/+5 |
| | | | |||||
| * | | Allow adding custom paths for action_cable channels | Sergey Novikov | 2016-01-22 | 1 | -2/+4 |
| | | | |||||
* | | | Ditch the EM error logging helper | Matthew Draper | 2016-01-24 | 1 | -8/+0 |
| | | | | | | | | | | | | | | | | | | 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. | ||||
* | | | Import the relevant portions of faye-websocket | Matthew Draper | 2016-01-24 | 7 | -21/+321 |
| | | | | | | | | | | | | (as adapted to use concurrent-ruby / nio4r instead of eventmachine) | ||||
* | | | Using a hacked faye-websocket, drop EventMachine | Matthew Draper | 2016-01-24 | 9 | -22/+33 |
| | | | |||||
* | | | Allow subscription adapters to be shut down | Matthew Draper | 2016-01-24 | 4 | -14/+40 |
| | | | |||||
* | | | Add Async and Inline adapters | Matthew Draper | 2016-01-24 | 2 | -0/+44 |
| | | | | | | | | | | | | | | | Just like their ActiveJob equivalents, these only work within the current process. | ||||
* | | | Normalize on no arguments for the success callback | Matthew Draper | 2016-01-24 | 2 | -2/+2 |
| | | | |||||
* | | | Split internal subscriber tracking from Postgres adapter | Matthew Draper | 2016-01-24 | 3 | -24/+70 |
| | | | |||||
* | | | Use the correct reference to the redis connection | Brit Gardner | 2016-01-22 | 1 | -1/+1 |
|/ / | | | | | | | | | Fixes #23193 This was throwing `There was an exception - NameError(undefined local variable or method `hi_redis_conn' for #<ActionCable::SubscriptionAdapter::Redis:0x007fb1449e2b70>)` on unsubscribe. | ||||
* | | Merge pull request #22950 from maclover7/adapterize-storage-actioncable | Matthew Draper | 2016-01-20 | 13 | -42/+204 |
|\ \ | | | | | | | | | | Adapterize storage for ActionCable | ||||
| * | | Fix code review comments | Jon Moss | 2016-01-18 | 9 | -46/+42 |
| | | | | | | | | | | | | | | | | | | | | | - adapter -> pubsub (re)rename internally - Change variable names to match method names - Add EventMachine `~> 1.0` as a runtime dependency of ActionCable - Refactor dependency loading for adapters | ||||
| * | | Small PostgreSQL adapter refactors / cleanup | Jon Moss | 2016-01-18 | 2 | -15/+16 |
| | | | | | | | | | | | | | | | - Escape the channel name when subscribing in PG - Refactor popping the queue to make it easier to read | ||||
| * | | ActionCable::StorageAdapter ==> ActionCable::SubscriptionAdapter | Jon Moss | 2016-01-18 | 7 | -14/+16 |
| | | | |||||
| * | | All Redis deps are now optional, Postgres --> PostgreSQL adapter | Jon Moss | 2016-01-18 | 3 | -4/+14 |
| | | | |||||
| * | | config_opts => cable, per @kaspth | Jon Moss | 2016-01-18 | 3 | -6/+7 |
| | | | |||||
| * | | Listener no longer needs to be a singleton | Matthew Draper | 2016-01-18 | 1 | -52/+58 |
| | | | | | | | | | | | | | | | | | | | | | We now only create one adapter instance for the server, so it can hold the listener. This in turn allows the listener to get the PG connection from the adapter, which will be a good place to allow more flexible configuration. | ||||
| * | | Tests passing and small refactoring | Jon Moss | 2016-01-18 | 1 | -1/+1 |
| | | | |||||
| * | | Blanket unsubscribe doesn't appear to be used | Matthew Draper | 2016-01-18 | 1 | -6/+2 |
| | | | |||||
| * | | Don't execute callbacks on our main listener thread | Matthew Draper | 2016-01-18 | 1 | -1/+1 |
| | | | |||||
| * | | Pull the action methods directly onto the adapter | Matthew Draper | 2016-01-18 | 8 | -56/+51 |
| | | | |||||
| * | | Add Postgres adapter | Jon Moss | 2016-01-18 | 2 | -0/+110 |
| | | | |||||
| * | | Refactor storage_adapter | Jon Moss | 2016-01-18 | 1 | -11/+3 |
| | | | |||||
| * | | Adapterize ActionCable storage and extract behavior | Jon Moss | 2016-01-18 | 11 | -38/+93 |
|/ / | |||||
* | | Fix the updated API (this sorts the concurrent-ruby switch with Basecamp) | David Heinemeier Hansson | 2016-01-16 | 1 | -1/+1 |
| | | |||||
* | | Revert "Merge pull request #22977 from rails/revert-22934-master" | David Heinemeier Hansson | 2016-01-16 | 5 | -21/+44 |
| | | | | | | | | | | This reverts commit d0393fccffc118a5de37654aa222774b66123393, reversing changes made to 3b7ccadfc1c8dfec61af898167e1300b17f5cf25. | ||||
* | | [ci skip] Fix typo in docs | DAVID MOORE | 2016-01-15 | 1 | -1/+1 |
| | | | | | | | | | | Found this small omission while reviewing the ActionCable docs. [Dave Moore] | ||||
* | | Revert "Move async execution from celluloid to concurrent-ruby" | David Heinemeier Hansson | 2016-01-08 | 5 | -44/+21 |
| | | |||||
* | | don't need explicit dep and a pretty neat pick | Mike Perham | 2016-01-05 | 1 | -1/+1 |
| | | |||||
* | | Use Module#thread_mattr_accessor | Mike Perham | 2016-01-05 | 1 | -8/+6 |
| | | |||||
* | | Move async execution from celluloid to concurrent-ruby | Mike Perham | 2016-01-05 | 5 | -21/+46 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | This removes 8 runtime gem dependencies from Rails: ``` Using hitimes 1.2.3 Using timers 4.1.1 Using celluloid-essentials 0.20.5 Using celluloid-extras 0.20.5 Using celluloid-fsm 0.20.5 Using celluloid-pool 0.20.5 Using celluloid-supervision 0.20.5 Using celluloid 0.17.2 ``` | ||||
* | | [ActionCable] remove not needed protected call and newlines | Jan Habermann | 2015-12-25 | 2 | -9/+1 |
|/ | |||||
* | Change `alpha` to `beta1` to prep for release of Rails 5 | eileencodes | 2015-12-18 | 1 | -1/+1 |
| | | | | :tada: :beers: | ||||
* | [ci skip] Fix grammar | Akshay Vishnoi | 2015-12-18 | 2 | -2/+2 |
| | |||||
* | [ci skip] Fix typo | Akshay Vishnoi | 2015-12-18 | 1 | -1/+1 |
| | |||||
* | Merge pull request #22639 from arunagw/aa-remove-few-warnings-actioncable | Guillermo Iguaran | 2015-12-17 | 5 | -4/+11 |
|\ | | | | | Remove warnings from actioncable test | ||||
| * | Remove warnings from actioncable | Arun Agrawal | 2015-12-17 | 5 | -4/+11 |
| | | | | | | | | Warnings coming from code and test are removed | ||||
* | | Don't log Action Cable to STDOUT in development | eileencodes | 2015-12-17 | 2 | -10/+0 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Logging Action Cable to STDOUT caused the development log to see double messages like this: ``` Started GET "/" for ::1 at 2015-12-17 15:21:34 -0500 Started GET "/" for ::1 at 2015-12-17 15:21:34 -0500 Processing by Rails::WelcomeController#index as HTML Processing by Rails::WelcomeController#index as HTML Rendered /welcome/index.html.erb (0.0ms) Rendered /welcome/index.html.erb (0.0ms) Completed 200 OK in 3ms (Views: 1.3ms | ActiveRecord: 0.0ms) Completed 200 OK in 3ms (Views: 1.3ms | ActiveRecord: 0.0ms) ``` Now that Action Cable is part of Rails it doesn't need it's own logger and will log to STDOUT via the local dev server here: https://github.com/rails/rails/blob/master/railties/lib/rails/commands/server.rb | ||||
* | | traditional make sense with `<tt>ActiveSupport::TaggedLogging</tt>` | Gaurav Sharma | 2015-12-18 | 1 | -2/+2 |
|/ | |||||
* | Cargo cult fix to make Celluloid behave | David Heinemeier Hansson | 2015-12-16 | 1 | -0/+3 |
| | |||||
* | Another stab at only mounting redis if the config is there | David Heinemeier Hansson | 2015-12-16 | 1 | -2/+2 |
| | |||||
* | Assume that the cable server is running in-process by default, offer option ↵ | David Heinemeier Hansson | 2015-12-16 | 1 | -0/+2 |
| | | | | to change that in config/environments/production.rb |