Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Merge pull request #115 from habermann24/master | David Heinemeier Hansson | 2015-12-11 | 1 | -2/+2 |
|\ | | | | | Fix an error when using multiple gid identifiers | ||||
| * | Use to_gid_param for connection identifiers | Jan Habermann | 2015-11-18 | 1 | -2/+2 |
| | | |||||
| * | Fix an error when using multiple gid identifiers | Jan Habermann | 2015-11-05 | 1 | -1/+1 |
| | | |||||
* | | Handle cases where logger is not a tagged logger. | Lachlan Sylvester | 2015-11-18 | 3 | -8/+14 |
| | | | | | | | | | | | | | | | | | | | | | | Previously, a TaggedLoggerProxy was only created if the logger responded to :tagged, but was still used as if it was a TaggedLoggerProxy elsewhere in the code, causing undefined method errors. This moved the check for tagging abilities inside the TaggedLoggerProxy so the code can always tread the logger like a tagged logger, and if it is not a tagged logger the tags will just be ignored. This prevents needing to check if the logger is tagged every time we use it. | ||||
* | | Share internal identifiers and message types with the JavaScript client | Javan Makhmali | 2015-11-06 | 2 | -6/+3 |
|/ | |||||
* | Rename Subscription#reject! to Subscription#reject as there's only one ↵ | Pratik Naik | 2015-11-05 | 1 | -2/+2 |
| | | | | version of the method | ||||
* | Add some documentation explaining subscription rejection | Pratik Naik | 2015-11-04 | 1 | -0/+16 |
| | |||||
* | Fix a merge fail syntax issue | Pratik Naik | 2015-11-04 | 1 | -1/+1 |
| | |||||
* | Merge branch 'master' into subscription-rejection | Pratik Naik | 2015-11-04 | 4 | -38/+39 |
|\ | |||||
| * | Make sure cable closes the connection if open when responding to an invalid ↵ | Pratik Naik | 2015-11-04 | 1 | -1/+2 |
| | | | | | | | | request | ||||
| * | 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 |
| | | | | |||||
* | | | | Allow rejecting subscriptions from the channel | Pratik Naik | 2015-10-21 | 3 | -8/+34 |
|/ / / | |||||
* | | | 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 | 2 | -3/+31 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 |
| | | | |||||
* | | | Devolve blanket #require to reveal intent and responsibility | Jeremy Daer | 2015-10-16 | 14 | -47/+85 |
| | | | | | | | | | | | | | | | | | | | | | | | | * 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 | 2 | -0/+21 |
| | | | |||||
* | | | 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 | 1 | -1/+1 |
| | | | | |||||
| * | | | First take at cross site forgery protection | Pratik Naik | 2015-10-07 | 2 | -1/+26 |
| |/ / | |||||
* | | | 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 |
|/ / | |||||
* / | support connection identifiers that don't implement to_global_id by ↵ | Mark Humphreys | 2015-08-24 | 1 | -1/+1 |
|/ | | | | defaulting to to_s | ||||
* | 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 | 1 | -0/+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 | 1 | -0/+2 |
| | | | | | | | | where it is needed. | ||||
* | | Merge pull request #50 from lsylvester/fix-stream-for-documentation | Kasper Timm Hansen | 2015-08-02 | 1 | -1/+1 |
|\ \ | |/ |/| | update docs for broadcast_to to pass a message. | ||||
| * | update docs for broadcast_to to pass a message to broadcast. | Lachlan Sylvester | 2015-08-02 | 1 | -1/+1 |
| | | |||||
* | | Fixing some documentation, correcting grammar, and removing unnecessary ↵ | Jason Dew | 2015-07-30 | 5 | -16/+16 |
|/ | | | | whitespace | ||||
* | refactor channel look up to use a hash instead of an array and reduce the ↵ | Lachlan Sylvester | 2015-07-30 | 2 | -5/+3 |
| | | | | number of calls to safe_constantize because it can be slow | ||||
* | Merge pull request #34 from ↵ | Kasper Timm Hansen | 2015-07-29 | 7 | -11/+85 |
|\ | | | | | | | | | lsylvester/add-broadcast_to-and-stream_for-methods-to-channel add broadcast_to and stream_for methods as per #26 | ||||
| * | add broadcast_to and stream_for methods as per #26 | Lachlan Sylvester | 2015-07-28 | 7 | -11/+85 |
| | |