aboutsummaryrefslogtreecommitdiffstats
path: root/lib
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #73 from smellsblue/websocket-url-differentiationDavid Heinemeier Hansson2015-12-134-1/+40
|\ | | | | Easy websocket url configuration
| * Easy websocket url configurationMike Virata-Stone2015-12-114-1/+40
| | | | | | | | Thanks to feedback from @ascrazy, @jeremy, and @javan
* | 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-068-27/+30
| | |
* | | Missed updating a method namePratik Naik2015-11-061-1/+1
|/ /
* | 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
| | | | |
* | | | | Better method names in Javascript based on the feedback from @javanPratik Naik2015-10-222-18/+19
| | | | |
* | | | | Allow rejecting subscriptions from the channelPratik Naik2015-10-216-10/+50
|/ / / /
* | | | 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-165-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 requireJeremy Daer2015-10-161-0/+4
| | | |
* | | | Merge branch 'adjust-reconnect'Jeremy Daer2015-10-162-23/+29
|\ \ \ \
| * | | | Tweak reconnect timingJavan Makhmali2015-10-141-14/+15
| | | | |
| * | | | Remove immediate reconnect to help avoid thundering herd after server restartsJavan Makhmali2015-10-141-4/+0
| | | | |
| * | | | Avoid returning results of loopJavan Makhmali2015-10-141-0/+1
| | | | |
| * | | | Add delay before reopeningJavan Makhmali2015-10-141-2/+9
| | | | |
| * | | | Improve guard against opening multiple web socketsJavan Makhmali2015-10-141-3/+4
| | | | |
* | | | | Devolve blanket #require to reveal intent and responsibilityJeremy Daer2015-10-1615-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 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-153-0/+23
|/ / / /
* | | | 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-072-2/+1
| | | | |
| * | | | First take at cross site forgery protectionPratik Naik2015-10-073-1/+27
| |/ / /
* | | | 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
| | | |