Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Fixing subscription callbacks | Diego Ballona | 2015-11-04 | 1 | -3/+6 |
| | |||||
* | Merge pull request #69 from mieko/callbacks | Rafael Mendonça França | 2015-10-23 | 3 | -32/+27 |
|\ | | | | | | | Use ActiveSupport::Callbacks for Channel subscription callbacks. | ||||
| * | Use ActiveSupport::Callbacks for Channel subscription callbacks. | Mike A. Owens | 2015-08-24 | 3 | -31/+32 |
| | | | | | | | | | | | | | | | | | | | | | | | | * Rely on AS::Callbacks for callback handling. * Add before_subscribe, after_subscribe, before_unsubscribe and after_unsubscribe convenience methods * alias on_subscribe and on_unsubscribe to after_subscribe and after_unsubscribe, respectively. * Remove `subscribed` and `unsubscribed` from the callback chain: these methods are now executed as the subject of the callbacks. * Update portions of ActionCable to use the more specific callback names. | ||||
* | | Merge pull request #84 from alexpeattie/nonstandard-logger-fix | Rafael França | 2015-10-23 | 1 | -3/+5 |
|\ \ | | | | | | | Fix NoMethodError when using a custom Rails.logger class | ||||
| * | | Fix NoMethodError when using a custom Rails.logger class | Alex Peattie | 2015-10-06 | 1 | -3/+5 |
| | | | |||||
* | | | Make sure the subscription confirmaion is only sent out once | Pratik Naik | 2015-10-19 | 1 | -2/+10 |
| | | | |||||
* | | | Freeze the SUBSCRIPTION_CONFIRMATION_INTERNAL_MESSAGE value | Pratik Naik | 2015-10-19 | 1 | -1/+1 |
| | | | |||||
* | | | Send subscription confirmation from server to the client to avoid race ↵ | Pratik Naik | 2015-10-16 | 5 | -9/+43 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | conditions. Without this, it's very easy to send messages over a subscription even before the redis pubsub has been fully initialized. Now we delay calling the subscription#connected method on the client side until we receive a subscription confirmation message from the server. | ||||
* | | | Cover stray deps for the logging convenience require | Jeremy Daer | 2015-10-16 | 1 | -0/+4 |
| | | | |||||
* | | | Merge branch 'adjust-reconnect' | Jeremy Daer | 2015-10-16 | 2 | -23/+29 |
|\ \ \ | |||||
| * | | | Tweak reconnect timing | Javan Makhmali | 2015-10-14 | 1 | -14/+15 |
| | | | | |||||
| * | | | Remove immediate reconnect to help avoid thundering herd after server restarts | Javan Makhmali | 2015-10-14 | 1 | -4/+0 |
| | | | | |||||
| * | | | Avoid returning results of loop | Javan Makhmali | 2015-10-14 | 1 | -0/+1 |
| | | | | |||||
| * | | | Add delay before reopening | Javan Makhmali | 2015-10-14 | 1 | -2/+9 |
| | | | | |||||
| * | | | Improve guard against opening multiple web sockets | Javan Makhmali | 2015-10-14 | 1 | -3/+4 |
| | | | | |||||
* | | | | Devolve blanket #require to reveal intent and responsibility | Jeremy Daer | 2015-10-16 | 15 | -71/+96 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Move specific requires close to where they're needed. * Use the private active_support/rails dep to wrap up common needs like eager autoload and module delegation. * Use a single Rails engine rather than an engine and a railtie. * Prefer `AS::JSON.encode` to `Object#to_json`. | ||||
* | | | | Test against Rails edge by default. CI against 4.2 also. | Jeremy Daer | 2015-10-16 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | * Don't deep-require to AD::Http::Request since it misses Mime autoload | ||||
* | | | | Shush some low-hanging Ruby warnings | Jeremy Daer | 2015-10-16 | 2 | -2/+2 |
| | | | | |||||
* | | | | Websockets -> WebSocket spelling [ci skip] | Jeremy Daer | 2015-10-15 | 4 | -16/+16 |
| | | | | |||||
* | | | | Fix the variable name in error message to make sure it does not raise an ↵ | Pratik Naik | 2015-10-15 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | exception | ||||
* | | | | Subscribe and unsubscribe from the internal redis channels in the primary EM ↵ | Pratik Naik | 2015-10-14 | 1 | -2/+2 |
| | | | | | | | | | | | | | | | | thread | ||||
* | | | | Don't add the current connection to the connections array until after all ↵ | Pratik Naik | 2015-10-14 | 1 | -2/+1 |
| | | | | | | | | | | | | | | | | the callbacks are run | ||||
* | | | | Remove some excessive logging | Pratik Naik | 2015-10-14 | 2 | -3/+0 |
| | | | | |||||
* | | | | add railtie and set default allowed_request_origins for development | Lachlan Sylvester | 2015-10-15 | 3 | -0/+23 |
|/ / / | |||||
* | | | Merge pull request #85 from rails/cross-site-forgery-protection | Pratik | 2015-10-12 | 2 | -1/+15 |
|\ \ \ | | | | | | | | | Add support for cross site forgery protection | ||||
| * | | | Treat ORIGIN as an opaque identifier and do equality comparison with the ↵ | Pratik Naik | 2015-10-12 | 1 | -14/+3 |
| | | | | | | | | | | | | | | | | specified whitelist | ||||
| * | | | Use Array() instead of Array.wrap | Pratik Naik | 2015-10-07 | 2 | -2/+1 |
| | | | | |||||
| * | | | First take at cross site forgery protection | Pratik Naik | 2015-10-07 | 3 | -1/+27 |
| |/ / | |||||
* | | | Make sure active record queries are logged with the current connection tags | Pratik Naik | 2015-10-12 | 4 | -5/+13 |
| | | | |||||
* | | | Be sure not to cache an empty string as the connection_identifier | Pratik Naik | 2015-10-09 | 1 | -1/+5 |
| | | | |||||
* | | | Include request id in statistics to make it to search the logs | Pratik Naik | 2015-10-09 | 1 | -1/+6 |
| | | | |||||
* | | | Better schedule pubsub subscribe | Pratik Naik | 2015-10-09 | 1 | -1/+1 |
| | | | |||||
* | | | Be sure to initialize @heartbeat_timer. Third time is a charm! | Pratik Naik | 2015-10-08 | 1 | -2/+2 |
| | | | |||||
* | | | Setup the heartbeat_timer in next tick to make sure EM reactor loop is running | Pratik Naik | 2015-10-08 | 1 | -2/+4 |
| | | | |||||
* | | | Run a single eventmachine timer to send heartbeats | Pratik Naik | 2015-10-08 | 5 | -35/+18 |
|/ / | |||||
* | | EventMachine: shush epoll warnings by checking for support before enabling. ↵ | Jeremy Daer | 2015-09-02 | 1 | -1/+2 |
| | | | | | | | | Ditto for kqueue. | ||||
* | | .js.coffee -> .coffee | Rafael Mendonça França | 2015-09-02 | 6 | -0/+0 |
| | | | | | | | | | | | | | | It was initially required, but support for the shorthand has been supported since sprockets 2.1. Eventually 4.x will only support the shorthand version. Just want to get new people using the prefer stuff ASAP. | ||||
* | | Increment style | Javan Makhmali | 2015-08-31 | 1 | -3/+2 |
| | | |||||
* | | Guard against opening multiple connections | Javan Makhmali | 2015-08-31 | 1 | -1/+1 |
| | | |||||
* | | Immediately reconnect after first disconnect | Javan Makhmali | 2015-08-28 | 1 | -0/+7 |
| | | |||||
* | | Only send "disconnected" notification once | Javan Makhmali | 2015-08-28 | 1 | -2/+8 |
| | | |||||
* | | Simplify WebSocket reconnects and guard against opening multiple connections | Javan Makhmali | 2015-08-28 | 1 | -27/+9 |
| | | |||||
* | | Merge pull request #65 from ↵ | Kasper Timm Hansen | 2015-08-24 | 1 | -1/+1 |
|\ \ | | | | | | | | | | | | | mrlhumphreys/support-identifiers-without-to-global-id Support connection identifiers that don't implement to_global_id | ||||
| * | | support connection identifiers that don't implement to_global_id by ↵ | Mark Humphreys | 2015-08-24 | 1 | -1/+1 |
| |/ | | | | | | | defaulting to to_s | ||||
* | | Record last 20 Subscription notifications for inspection | Javan Makhmali | 2015-08-23 | 1 | -1/+12 |
| | | |||||
* | | Immediately reconnect when visibilityState changes to "visible" | Javan Makhmali | 2015-08-23 | 1 | -0/+9 |
|/ | |||||
* | Remove out-of-date AC::Broadcaster reference | Kai Chen = 陈恺 | 2015-08-13 | 1 | -2/+0 |
| | | | Remove out-of-data autoload reference of ActionCable::Broadcaster that removed at e1a99a83ca135523ff8513be756f156500999cb8 . | ||||
* | Clear out the streams when they are stopped. Otherwise we will keep trying ↵ | Lachlan Sylvester | 2015-08-11 | 1 | -1/+1 |
| | | | | to stop them. | ||||
* | Merge pull request #54 from lsylvester/require-to_param | Kasper Timm Hansen | 2015-08-03 | 2 | -1/+2 |
|\ | | | | | Move require of to_param to where it is needed | ||||
| * | move the require of object/to_param to channel/broadcasting because that is ↵ | Lachlan Sylvester | 2015-08-02 | 2 | -1/+2 |
| | | | | | | | | where it is needed. |