aboutsummaryrefslogtreecommitdiffstats
path: root/actioncable
Commit message (Collapse)AuthorAgeFilesLines
...
* | | Shut down the worker pool - don't kill itMatthew Draper2016-10-031-1/+1
| | | | | | | | | | | | | | | Different parts of concurrent-ruby's documentation make inconsistent claims about how kill will behave. It doesn't do the thing we want.
* | | Add tests for Server::Base#restartJon Moss2016-10-021-0/+33
| | |
* | | Move behavior to Server::Base, and flush pubsubJon Moss2016-10-022-3/+6
| | |
* | | Shutdown pubsub connection before classes are reloadedJon Moss2016-10-021-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Before this patch, if you were to make a file edit in your Rails application and you tried to load up the page, it would hang indefinitely. The issue is that Active Record is trying to cleanup after itself and clear all active connection, but Action Cable is still holding onto a connection from the pool. To resolve this, we are now shutting down the pubsub adapter before classes are reloaded, to avoid this altogether (connection is being returned to the pool). Credits to @skateman for discovering this bug. :)
* | | Use websocket-client-simple instead of Faye as a websockets clientMatthew Draper2016-10-021-57/+81
| | | | | | | | | | | | | | | | | | Mostly, this is just to avoid EventMachine. But there's also an argument to be made that we're better off using a different protocol library for our test suite than the one we use to implement the server.
* | | Shut down EventMachine before re-enabling warningsMatthew Draper2016-10-022-0/+9
| | |
* | | In-line the configuration points that only existed for Faye supportMatthew Draper2016-10-015-13/+4
| | |
* | | Remove Faye modeMatthew Draper2016-10-0110-168/+7
| | | | | | | | | | | | No deprecation, because it was never documented.
* | | Merge pull request #26547 from ↵Matthew Draper2016-10-019-24/+83
|\ \ \ | | | | | | | | | | | | | | | | | | | | palkan/fix/actioncable-confirmation-race-condition Avoid race condition on subscription confirmation
| * | | Add Channel#ensure_confirmation_sent; call #subscribe_to_channel after ↵palkan2016-09-227-33/+39
| | | | | | | | | | | | | | | | initializing
| * | | [Fix #25381] Avoid race condition on subscription confirmationpalkan2016-09-206-15/+60
| | | |
* | | | Buffer writes to the cable socketsMatthew Draper2016-09-284-12/+100
| |/ / |/| | | | | | | | | | | Otherwise, they can sometimes block, leading to reduced system throughput.
* | | [ci skip] Fix formatting of documentation of worker_pool method from ↵Prathamesh Sonpatki2016-09-071-1/+1
| | | | | | | | | | | | AC::Server::Base
* | | [ci skip] Fix formatting in Action Cable Connection::Base module docsPrathamesh Sonpatki2016-09-071-1/+1
| | |
* | | [ci fix] Fix API documentation for Streams module from Action CablePrathamesh Sonpatki2016-09-061-4/+4
|/ /
* | fixes remaining RuboCop issues [Vipul A M, Xavier Noria]Xavier Noria2016-09-012-2/+2
| |
* | Prevent invocation of channel action if rejected connectionJon Moss2016-08-192-1/+16
| | | | | | | | | | | | | | | | | | Fixes #23757. Before this commit, even if `reject` was called in the `subscribe` method for an Action Cable channel, all actions on that channel could still be invoked. This calls a `return` if a rejected connection tries to invoke any actions on the channel.
* | Add three new rubocop rulesRafael Mendonça França2016-08-163-26/+26
| | | | | | | | | | | | | | | | Style/SpaceBeforeBlockBraces Style/SpaceInsideBlockBraces Style/SpaceInsideHashLiteralBraces Fix all violations in the repository.
* | Pass over changelogs [ci skip]Vipul A M2016-08-101-1/+1
| |
* | revises more Lint/EndAlignment offensesXavier Noria2016-08-081-6/+6
| |
* | copy-edits an exception messageXavier Noria2016-08-071-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Inserted spaces in the name of Rails components. Since I was on it, also used PostgreSQL instead of Postgres because albeit Postgres is an accepted alias, PostgreSQL is the official name and the actual name of the adapter. See https://wiki.postgresql.org/wiki/ProjectName with regard to PostgreSQL vs Postgres.
* | Add `Style/EmptyLines` in `.rubocop.yml` and remove extra empty linesRyuta Kamizono2016-08-072-3/+0
| |
* | applies remaining conventions across the projectXavier Noria2016-08-062-4/+3
| |
* | normalizes indentation and whitespace across the projectXavier Noria2016-08-062-4/+4
| |
* | remove redundant curlies from hash argumentsXavier Noria2016-08-065-7/+7
| |
* | modernizes hash syntax in actioncableXavier Noria2016-08-061-2/+2
| |
* | applies new string literal convention to the rest of the projectXavier Noria2016-08-061-6/+6
| | | | | | | | | | The current code base is not uniform. After some discussion, we have chosen to go with double quotes by default.
* | applies new string literal convention in the gemspecsXavier Noria2016-08-061-14/+14
| | | | | | | | | | The current code base is not uniform. After some discussion, we have chosen to go with double quotes by default.
* | applies new string literal convention in actioncable/testXavier Noria2016-08-0629-260/+260
| | | | | | | | | | The current code base is not uniform. After some discussion, we have chosen to go with double quotes by default.
* | applies new string literal convention in actioncable/libXavier Noria2016-08-0628-90/+90
| | | | | | | | | | The current code base is not uniform. After some discussion, we have chosen to go with double quotes by default.
* | Give importance to rails command in the comment section.Santosh Wadghule2016-07-131-1/+1
| |
* | Correct changelog layoutMatthew Draper2016-07-091-8/+8
| | | | | | | | [ci skip]
* | Insert changelog entry for #25615Matthew Draper2016-07-091-0/+7
| |
* | Merge pull request #25624 from tinco/actioncable_write_raceMatthew Draper2016-07-092-2/+12
|\ \ | | | | | | | | | Fix race condition in websocket stream write
| * | fix race condition in websocket stream writeTinco Andringa2016-07-011-0/+4
| | |
* | | Merge pull request #24988 from mwear/action_cable_broadcast_notificationsMatthew Draper2016-07-043-6/+58
|\ \ \ | | | | | | | | Add ActiveSupport::Notifications hook to ActionCable::Server.broadcast
| * | | Add ActiveSupport::Notifications hook to Broadcaster#broadcastMatthew Wear2016-05-123-6/+58
| | | | | | | | | | | | | | | | | | | | This addition of this notification hook will give users better visibility into the messages being sent over the PubSub adapter.
* | | | Merge pull request #25030 from mmmpa/pull_requestMatthew Draper2016-07-042-1/+22
|\ \ \ \ | | | | | | | | | | ActionCable, sometimes add_channel is not called.
| * | | | Add guard to broadcast.mmmpa2016-05-262-1/+22
| | | | |
* | | | | Merge pull request #25293 from y-yagi/remove_needless_commentsMatthew Draper2016-07-033-5/+0
|\ \ \ \ \ | |_|_|_|/ |/| | | | remove needless comments
| * | | | remove needless commentsyuuji.yaginuma2016-06-063-5/+0
| | | | | | | | | | | | | | | | | | | | Follow up to #25240.
* | | | | close hijacked i/o socket after use (fixes #25613)Tinco Andringa2016-07-012-0/+15
| |_|_|/ |/| | |
* | | | Periodic timers: delegate async instance_exec to the worker poolJeremy Daer2016-06-292-7/+9
| | | |
* | | | Merge pull request #24991 from maclover7/actioncable-npm-docsVipul A M2016-06-291-0/+68
|\ \ \ \ | | | | | | | | | | Add documentation about Action Cable npm package
| * | | | Add documentation about Action Cable npm packageJon Moss2016-05-261-0/+68
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | Sorry, forgot to include in my main PR :( [ci skip] [Jon Moss, Zach Schneider]
* | | | Make client test run individually.Kasper Timm Hansen2016-06-281-1/+3
| | | | | | | | | | | | | | | | Require our dependency and forgo the core ext version.
* | | | Fix Action Cable README [ci skip]Rafael Mendonça França2016-06-261-5/+14
| | | |
* | | | Remove conflict line [ci skip]Ryuta Kamizono2016-06-271-1/+0
| | | |
* | | | Merge branch 'master' of github.com:rails/docrailsVijay Dev2016-06-261-2/+143
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | Conflicts: actioncable/README.md
| * | | | Fix and make example generic in README as well [ci skip]Vipul A M2016-06-151-3/+3
| | | | |