Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
| * | | | | | Freeze the SUBSCRIPTION_CONFIRMATION_INTERNAL_MESSAGE value | Pratik Naik | 2015-10-19 | 1 | -1/+1 | |
| | | | | | | ||||||
| * | | | | | Fix an error message in the subscription tests | Pratik Naik | 2015-10-16 | 1 | -1/+1 | |
| | | | | | | ||||||
| * | | | | | Send subscription confirmation from server to the client to avoid race ↵ | Pratik Naik | 2015-10-16 | 8 | -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 tests | Pratik Naik | 2015-10-16 | 1 | -2/+2 | |
| | | | | | ||||||
* | | | | | 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 | |
| | | | | | | ||||||
* | | | | | | Merge pull request #97 from lsylvester/speed-up-tests | Jeremy Daer | 2015-10-16 | 1 | -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 timer | Lachlan Sylvester | 2015-10-17 | 1 | -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 responsibility | Jeremy Daer | 2015-10-16 | 18 | -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 Daer | 2015-10-16 | 6 | -38/+84 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Don't deep-require to AD::Http::Request since it misses Mime autoload | |||||
* | | | | | | | Shush some low-hanging Ruby warnings | Jeremy Daer | 2015-10-16 | 5 | -4/+14 | |
|/ / / / / / | ||||||
* | | | | | | gemspec: prefer requiring from head of the load path | Jeremy Daer | 2015-10-15 | 1 | -1/+1 | |
| | | | | | | ||||||
* | | | | | | README was moved to README.md at a9c3fd5 | Jeremy Daer | 2015-10-15 | 1 | -1/+1 | |
| | | | | | | ||||||
* | | | | | | Websockets -> WebSocket spelling [ci skip] | Jeremy Daer | 2015-10-15 | 7 | -28/+28 | |
| | | | | | | ||||||
* | | | | | | Run connection tests in EM loop | Pratik Naik | 2015-10-15 | 10 | -128/+242 | |
| | | | | | | ||||||
* | | | | | | 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 | |
| | | | | | | ||||||
* | | | | | | Merge pull request #93 from lsylvester/railstie | Kasper Timm Hansen | 2015-10-14 | 3 | -0/+23 | |
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | add railtie and set default allowed_request_origins for development | |||||
| * | | | | | add railtie and set default allowed_request_origins for development | Lachlan Sylvester | 2015-10-15 | 3 | -0/+23 | |
|/ / / / / | ||||||
* | | | | | Merge pull request #89 from kayakyakr/readme-improvements | David Heinemeier Hansson | 2015-10-13 | 1 | -15/+97 | |
|\ \ \ \ \ | | | | | | | | | | | | | Adding some examples to the README. | |||||
| * | | | | | splitting out a bit of ruby from some coffeescript that it was grouped with | Charles DuBose | 2015-10-12 | 1 | -4/+6 | |
| | | | | | | ||||||
| * | | | | | Adding some examples to the README. Since ActionCable is an engine, it can ↵ | Charles DuBose | 2015-10-12 | 1 | -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-protection | Pratik | 2015-10-12 | 4 | -2/+74 | |
|\ \ \ \ \ | | | | | | | | | | | | | Add support for cross site forgery protection | |||||
| * | | | | | Treat ORIGIN as an opaque identifier and do equality comparison with the ↵ | Pratik Naik | 2015-10-12 | 3 | -21/+11 | |
| | | | | | | | | | | | | | | | | | | | | | | | | specified whitelist | |||||
| * | | | | | Set appropriate origin and host in the tests | Pratik Naik | 2015-10-12 | 1 | -2/+3 | |
| | | | | | | ||||||
| * | | | | | 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 | 5 | -1/+83 | |
| | |/ / / | |/| | | | ||||||
* | | | | | 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 | 6 | -38/+19 | |
|/ / / / | ||||||
* | | | | Merge pull request #83 from gregmolnar/readme_fix | Eileen M. Uchitelle | 2015-10-03 | 1 | -1/+1 | |
|\ \ \ \ | | | | | | | | | | | use correct filename in example | |||||
| * | | | | use correct filename in example | Greg Molnar | 2015-10-03 | 1 | -1/+1 | |
|/ / / / | | | | | | | | | | | | | closes #80 | |||||
* | | | | EventMachine: shush epoll warnings by checking for support before enabling. ↵ | Jeremy Daer | 2015-09-02 | 1 | -1/+2 | |
| | | | | | | | | | | | | | | | | Ditto for kqueue. | |||||
* | | | | Merge pull request #71 from rails/coffee-only | Rafael Mendonça França | 2015-09-02 | 6 | -0/+0 | |
|\ \ \ \ | | | | | | | | | | | .js.coffee -> .coffee | |||||
| * | | | | .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. | |||||
* | | | | Merge branch 'better-reconnect' | Javan Makhmali | 2015-09-01 | 2 | -30/+24 | |
|\ \ \ \ | ||||||
| * | | | | 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 | |
| | | | | |