Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Use a gentler disconnect | Matthew Draper | 2016-01-30 | 1 | -2/+2 |
| | | | | The detach used by close! seems to be making EM very sad on Travis. | ||||
* | Don't bother stopping EM between tests | Matthew Draper | 2016-01-30 | 1 | -38/+3 |
| | | | | | It's not strictly necessary, and maybe this will help with the current test failure. | ||||
* | remove require logging | Kesha Antonov | 2016-01-29 | 1 | -2/+0 |
| | |||||
* | Keep the socket reference after close | Matthew Draper | 2016-01-30 | 1 | -3/+2 |
| | | | | We may still try to send to it. | ||||
* | Reduce the client count, in hope of a more consistent test | Matthew Draper | 2016-01-30 | 1 | -3/+3 |
| | |||||
* | Be more patient while gathering the expected responses | Matthew Draper | 2016-01-30 | 1 | -4/+3 |
| | |||||
* | Handle more IO errors (especially, ECONNRESET) | Matthew Draper | 2016-01-30 | 3 | -21/+78 |
| | | | | | | 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. | ||||
* | Add a couple of tests that connect with a WS client | Matthew Draper | 2016-01-30 | 2 | -0/+251 |
| | |||||
* | 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 | 25 | -99/+382 |
| | |||||
* | Revert "Eliminate the EventMachine dependency" | David Heinemeier Hansson | 2016-01-27 | 25 | -382/+99 |
| | |||||
* | [ci skip] Don’t explicitly mention EventMachine | claudiob | 2016-01-25 | 1 | -5/+4 |
| | | | | | | | | | | Since #23152 eliminated the EventMachine dependency, we don’t need to explicitly mention EventMachine. Nevertheless, I'm not 100% sure about saying "the websocket-driver loop" driver… any suggestions, @matthewd or @pixeltrix ? :sweat_smile: [ci skip] | ||||
* | 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 | 2 | -9/+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 | 12 | -30/+331 |
| | | | | | | | | | | | | (as adapted to use concurrent-ruby / nio4r instead of eventmachine) | ||||
* | | | Using a hacked faye-websocket, drop EventMachine | Matthew Draper | 2016-01-24 | 15 | -65/+56 |
| | | | |||||
* | | | Add tests for the ActionCable adapters | Matthew Draper | 2016-01-24 | 7 | -0/+227 |
| | | | |||||
* | | | 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 |
| | | | |||||
* | | | fix path of Action Cable configuration file [ci skip] | yuuji.yaginuma | 2016-01-23 | 1 | -2/+2 |
| | | | | | | | | | | | | This changed in #22950. | ||||
* | | | Move ApplicationCable::Channel section of readme | Derek Kniffin | 2016-01-22 | 1 | -7/+7 |
| | | | | | | | | | I believe the paragraph starting with "This relies" belongs with the one above it. | ||||
* | | | 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. | ||||
* | | Small doc update per RF [ci skip] | Jon Moss | 2016-01-20 | 1 | -4/+5 |
| | | |||||
* | | Add CHANGELOG.md entry for #22950 | Jon Moss | 2016-01-20 | 1 | -0/+8 |
| | | | | | | | | [ci skip] | ||||
* | | Merge pull request #22950 from maclover7/adapterize-storage-actioncable | Matthew Draper | 2016-01-20 | 22 | -56/+311 |
|\ \ | | | | | | | | | | Adapterize storage for ActionCable | ||||
| * | | Fix code review comments | Jon Moss | 2016-01-18 | 14 | -55/+52 |
| | | | | | | | | | | | | | | | | | | | | | - 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 | 11 | -21/+23 |
| | | | |||||
| * | | All Redis deps are now optional, Postgres --> PostgreSQL adapter | Jon Moss | 2016-01-18 | 6 | -9/+17 |
| | | | |||||
| * | | 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 | 7 | -37/+65 |
| | | | |||||
| * | | 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 | 3 | -0/+111 |
| | | | |||||
| * | | Refactor storage_adapter | Jon Moss | 2016-01-18 | 1 | -11/+3 |
| | | | |||||
| * | | Adapterize ActionCable storage and extract behavior | Jon Moss | 2016-01-18 | 13 | -38/+158 |
|/ / | |||||
* | | Merge pull request #23096 from HayleyCAnderson/ha-action-cable-docs | Jon Moss | 2016-01-18 | 1 | -20/+31 |
|\ \ | | | | | | | [ci skip] Improve Action Cable configuration explanations | ||||
| * | | Improve Action Cable configuration explanations | Hayley Anderson | 2016-01-18 | 1 | -20/+31 |
| | | | | | | | | | | | | | | | | | | | | | | | | * Clarify wording around how defaults work * Clarify wording around what formatting versus information is required * Separate consumer configuration section since it is important and wordy * Add additional explanation of options for setting consumer configuration * Consolidate consumer configuration information to reduce confusion | ||||
* | | | 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 | 14 | -35/+45 |
| | | | | | | | | | | | | | | | 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] |