aboutsummaryrefslogtreecommitdiffstats
path: root/actioncable
Commit message (Collapse)AuthorAgeFilesLines
...
* | | 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
| | | | |
* | | | | client test have not using Pathname and hashwithindifferent accessGaurav Sharma2016-06-261-3/+0
| | | | |
* | | | | Allow actioncable connections from any port in developmentMichael Dever2016-06-161-1/+1
| |_|/ / |/| | |
* | | | Added "Download and Installation" section to Action Cable README [ci skip]Prathamesh Sonpatki2016-06-041-0/+10
| | | | | | | | | | | | | | | | | | | | - Only Action Cable README did not have this section, all of the other READMEs have this section.
* | | | Trim Action Cable README [ci skip]Prathamesh Sonpatki2016-06-041-161/+0
|/ / / | | | | | | | | | | | | | | | - Remove "Configuration", "Running the server", "Dependencies" and "Deployment" sections from the Action Cable README as they are already duplicated in the Action Cable overview guide.