aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | Freeze the SUBSCRIPTION_CONFIRMATION_INTERNAL_MESSAGE valuePratik Naik2015-10-191-1/+1
| | | | | |
| * | | | | Fix an error message in the subscription testsPratik Naik2015-10-161-1/+1
| | | | | |
| * | | | | Send subscription confirmation from server to the client to avoid race ↵Pratik Naik2015-10-168-18/+76
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
* | | | | Fix stream testsPratik Naik2015-10-161-2/+2
| | | | |
* | | | | 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
| | | | | |
* | | | | | Merge pull request #97 from lsylvester/speed-up-testsJeremy Daer2015-10-161-1/+2
|\ \ \ \ \ \ | | | | | | | | | | | | | | Speed up tests be calling EM.run_deferred_callbacks instead of setting a timer
| * | | | | | Speed up tests be calling EM.run_deferred_callbacks instead of setting a timerLachlan Sylvester2015-10-171-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The run_in_eventmachine test helper method is setting a 0.1 second timer to stop the event machine loop. This causes each test that requires an event machine loop to wait for 0.1 second regardless of how long the test takes to process. This changes that to call EM.run_deferred_callbacks, which immediatly process pending actions in the event loop and then is able to exit the event loop without doing any waiting. Before this change, running tests produced Finished in 2.957857s, 15.8899 runs/s, 27.7228 assertions/s. After, the tests get Finished in 0.065942s, 712.7514 runs/s, 1243.5237 assertions/s.
* | | | | | | Devolve blanket #require to reveal intent and responsibilityJeremy Daer2015-10-1618-76/+103
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * 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-166-38/+84
| | | | | | | | | | | | | | | | | | | | | | | | | | | | * Don't deep-require to AD::Http::Request since it misses Mime autoload
* | | | | | | Shush some low-hanging Ruby warningsJeremy Daer2015-10-165-4/+14
|/ / / / / /
* | | | | | gemspec: prefer requiring from head of the load pathJeremy Daer2015-10-151-1/+1
| | | | | |
* | | | | | README was moved to README.md at a9c3fd5Jeremy Daer2015-10-151-1/+1
| | | | | |
* | | | | | Websockets -> WebSocket spelling [ci skip]Jeremy Daer2015-10-157-28/+28
| | | | | |
* | | | | | Run connection tests in EM loopPratik Naik2015-10-1510-128/+242
| | | | | |
* | | | | | 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
| | | | | |
* | | | | | Merge pull request #93 from lsylvester/railstieKasper Timm Hansen2015-10-143-0/+23
|\ \ \ \ \ \ | |/ / / / / |/| | | | | add railtie and set default allowed_request_origins for development
| * | | | | add railtie and set default allowed_request_origins for developmentLachlan Sylvester2015-10-153-0/+23
|/ / / / /
* | | | | Merge pull request #89 from kayakyakr/readme-improvementsDavid Heinemeier Hansson2015-10-131-15/+97
|\ \ \ \ \ | | | | | | | | | | | | Adding some examples to the README.
| * | | | | splitting out a bit of ruby from some coffeescript that it was grouped withCharles DuBose2015-10-121-4/+6
| | | | | |
| * | | | | Adding some examples to the README. Since ActionCable is an engine, it can ↵Charles DuBose2015-10-121-11/+91
|/ / / / / | | | | | | | | | | | | | | | be run in-app in the same way that websocket-rails can be (and I will admit to shamelessly stealing the implementation to websocket rails). Also adding examples of passing params to subscription and rebroadcasting received message.
* | | | | Merge pull request #85 from rails/cross-site-forgery-protectionPratik2015-10-124-2/+74
|\ \ \ \ \ | | | | | | | | | | | | Add support for cross site forgery protection
| * | | | | Treat ORIGIN as an opaque identifier and do equality comparison with the ↵Pratik Naik2015-10-123-21/+11
| | | | | | | | | | | | | | | | | | | | | | | | specified whitelist
| * | | | | Set appropriate origin and host in the testsPratik Naik2015-10-121-2/+3
| | | | | |
| * | | | | Use Array() instead of Array.wrapPratik Naik2015-10-072-2/+1
| | | | | |
| * | | | | First take at cross site forgery protectionPratik Naik2015-10-075-1/+83
| | |/ / / | |/| | |
* | | | | 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-086-38/+19
|/ / / /
* | | | Merge pull request #83 from gregmolnar/readme_fixEileen M. Uchitelle2015-10-031-1/+1
|\ \ \ \ | | | | | | | | | | use correct filename in example
| * | | | use correct filename in exampleGreg Molnar2015-10-031-1/+1
|/ / / / | | | | | | | | | | | | closes #80
* | | | EventMachine: shush epoll warnings by checking for support before enabling. ↵Jeremy Daer2015-09-021-1/+2
| | | | | | | | | | | | | | | | Ditto for kqueue.
* | | | Merge pull request #71 from rails/coffee-onlyRafael Mendonça França2015-09-026-0/+0
|\ \ \ \ | | | | | | | | | | .js.coffee -> .coffee
| * | | | .js.coffee -> .coffeeRafael Mendonça França2015-09-026-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.
* | | | Merge branch 'better-reconnect'Javan Makhmali2015-09-012-30/+24
|\ \ \ \
| * | | | Increment styleJavan Makhmali2015-08-311-3/+2
| | | | |
| * | | | Guard against opening multiple connectionsJavan Makhmali2015-08-311-1/+1
| | | | |
| * | | | Immediately reconnect after first disconnectJavan Makhmali2015-08-281-0/+7
| | | | |