aboutsummaryrefslogtreecommitdiffstats
path: root/lib/action_cable
Commit message (Collapse)AuthorAgeFilesLines
* Remove unused method allowed_origins in Connection::Baseadamliesko2015-12-131-5/+0
|
* Allow regexp for a allowed_request_origins arrayadamliesko2015-12-121-1/+6
|
* Merge pull request #57 from lsylvester/log-to-stdout-in-developement-modeDavid Heinemeier Hansson2015-12-112-0/+10
|\ | | | | Add logging to stdout in development mode.
| * log to stdout in development mode.Lachlan Sylvester2015-08-102-0/+10
| |
* | Merge pull request #115 from habermann24/masterDavid Heinemeier Hansson2015-12-111-2/+2
|\ \ | | | | | | Fix an error when using multiple gid identifiers
| * | Use to_gid_param for connection identifiersJan Habermann2015-11-181-2/+2
| | |
| * | Fix an error when using multiple gid identifiersJan Habermann2015-11-051-1/+1
| | |
* | | Handle cases where logger is not a tagged logger.Lachlan Sylvester2015-11-183-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 clientJavan Makhmali2015-11-062-6/+3
|/ /
* | Rename Subscription#reject! to Subscription#reject as there's only one ↵Pratik Naik2015-11-051-2/+2
| | | | | | | | version of the method
* | Add some documentation explaining subscription rejectionPratik Naik2015-11-041-0/+16
| |
* | Fix a merge fail syntax issuePratik Naik2015-11-041-1/+1
| |
* | Merge branch 'master' into subscription-rejectionPratik Naik2015-11-044-38/+39
|\ \
| * | Make sure cable closes the connection if open when responding to an invalid ↵Pratik Naik2015-11-041-1/+2
| | | | | | | | | | | | request
| * | Fixing subscription callbacksDiego Ballona2015-11-041-3/+6
| | |
| * | Merge pull request #69 from mieko/callbacksRafael Mendonça França2015-10-233-32/+27
| |\ \ | | | | | | | | | | | | Use ActiveSupport::Callbacks for Channel subscription callbacks.
| | * | Use ActiveSupport::Callbacks for Channel subscription callbacks.Mike A. Owens2015-08-243-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-fixRafael França2015-10-231-3/+5
| |\ \ \ | | | | | | | | | | Fix NoMethodError when using a custom Rails.logger class
| | * | | Fix NoMethodError when using a custom Rails.logger classAlex Peattie2015-10-061-3/+5
| | | | |
* | | | | Allow rejecting subscriptions from the channelPratik Naik2015-10-213-8/+34
|/ / / /
* | | | Make sure the subscription confirmaion is only sent out oncePratik Naik2015-10-191-2/+10
| | | |
* | | | Freeze the SUBSCRIPTION_CONFIRMATION_INTERNAL_MESSAGE valuePratik Naik2015-10-191-1/+1
| | | |
* | | | Send subscription confirmation from server to the client to avoid race ↵Pratik Naik2015-10-162-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 requireJeremy Daer2015-10-161-0/+4
| | | |
* | | | Devolve blanket #require to reveal intent and responsibilityJeremy Daer2015-10-1614-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 Daer2015-10-161-1/+1
| | | | | | | | | | | | | | | | * Don't deep-require to AD::Http::Request since it misses Mime autoload
* | | | Shush some low-hanging Ruby warningsJeremy Daer2015-10-162-2/+2
| | | |
* | | | Websockets -> WebSocket spelling [ci skip]Jeremy Daer2015-10-154-16/+16
| | | |
* | | | Fix the variable name in error message to make sure it does not raise an ↵Pratik Naik2015-10-151-1/+1
| | | | | | | | | | | | | | | | exception
* | | | Subscribe and unsubscribe from the internal redis channels in the primary EM ↵Pratik Naik2015-10-141-2/+2
| | | | | | | | | | | | | | | | thread
* | | | Don't add the current connection to the connections array until after all ↵Pratik Naik2015-10-141-2/+1
| | | | | | | | | | | | | | | | the callbacks are run
* | | | Remove some excessive loggingPratik Naik2015-10-142-3/+0
| | | |
* | | | add railtie and set default allowed_request_origins for developmentLachlan Sylvester2015-10-152-0/+21
| | | |
* | | | Merge pull request #85 from rails/cross-site-forgery-protectionPratik2015-10-122-1/+15
|\ \ \ \ | | | | | | | | | | Add support for cross site forgery protection
| * | | | Treat ORIGIN as an opaque identifier and do equality comparison with the ↵Pratik Naik2015-10-121-14/+3
| | | | | | | | | | | | | | | | | | | | specified whitelist
| * | | | Use Array() instead of Array.wrapPratik Naik2015-10-071-1/+1
| | | | |
| * | | | First take at cross site forgery protectionPratik Naik2015-10-072-1/+26
| |/ / /
* | | | Make sure active record queries are logged with the current connection tagsPratik Naik2015-10-124-5/+13
| | | |
* | | | Be sure not to cache an empty string as the connection_identifierPratik Naik2015-10-091-1/+5
| | | |
* | | | Include request id in statistics to make it to search the logsPratik Naik2015-10-091-1/+6
| | | |
* | | | Better schedule pubsub subscribePratik Naik2015-10-091-1/+1
| | | |
* | | | Be sure to initialize @heartbeat_timer. Third time is a charm!Pratik Naik2015-10-081-2/+2
| | | |
* | | | Setup the heartbeat_timer in next tick to make sure EM reactor loop is runningPratik Naik2015-10-081-2/+4
| | | |
* | | | Run a single eventmachine timer to send heartbeatsPratik Naik2015-10-085-35/+18
|/ / /
* / / support connection identifiers that don't implement to_global_id by ↵Mark Humphreys2015-08-241-1/+1
|/ / | | | | | | defaulting to to_s
* / Clear out the streams when they are stopped. Otherwise we will keep trying ↵Lachlan Sylvester2015-08-111-1/+1
|/ | | | to stop them.
* Merge pull request #54 from lsylvester/require-to_paramKasper Timm Hansen2015-08-031-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 Sylvester2015-08-021-0/+2
| | | | | | | | where it is needed.
* | Merge pull request #50 from lsylvester/fix-stream-for-documentationKasper Timm Hansen2015-08-021-1/+1
|\ \ | |/ |/| update docs for broadcast_to to pass a message.
| * update docs for broadcast_to to pass a message to broadcast.Lachlan Sylvester2015-08-021-1/+1
| |