Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
| * | | | Include changes made to the engine | David Heinemeier Hansson | 2015-12-16 | 1 | -0/+5 | |
| | | | | ||||||
| * | | | Move Cable to ActionCable for client-side constant to avoid conflicts | David Heinemeier Hansson | 2015-12-16 | 8 | -49/+49 | |
| | | | | ||||||
| * | | | We are using railtie, not engine | David Heinemeier Hansson | 2015-12-16 | 3 | -37/+2 | |
| | | | | ||||||
| * | | | Basic channel generator | David Heinemeier Hansson | 2015-12-15 | 7 | -0/+101 | |
| | | | | ||||||
| * | | | Move the require to the right place | Rafael Mendonça França | 2015-12-15 | 2 | -2/+1 | |
| | | | | ||||||
| * | | | Remove the default logger | Rafael Mendonça França | 2015-12-15 | 1 | -1/+0 | |
| | | | | | | | | | | | | | | | | It should be configured through the railtie | |||||
| * | | | Configure the Action Cable's redis in the engine | Rafael Mendonça França | 2015-12-15 | 3 | -23/+7 | |
| | | | | | | | | | | | | | | | | This will decouple Action Cable from Rails. | |||||
| * | | | Accept a Pathname in Application#config_for | Rafael Mendonça França | 2015-12-15 | 2 | -1/+20 | |
| | | | | | | | | | | | | | | | | That would make possible to use it with action cable configuration. | |||||
| * | | | Add Action Cable to README.md of Rails | claudiob | 2015-12-15 | 1 | -1/+3 | |
| | | | | | | | | | | | | | | | | [ci skip] Link to actioncable/README.md for more details | |||||
| * | | | Remove rails/actioncable Travis badge | claudiob | 2015-12-15 | 1 | -2/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [ci skip] Since Action Cable is getting integrated into rails/rails, the Travis badge for rails/actioncable is not needed anymore. Also following the standard of every other README to have `--` rather than `-` to separating the name of the library from its description. | |||||
| * | | | Merge remote-tracking branch 'origin/master' into merge-action-cable | Rafael Mendonça França | 2015-12-15 | 75 | -165/+419 | |
| |\ \ \ | ||||||
| * | | | | Action Cable need to be a engine | Rafael Mendonça França | 2015-12-15 | 2 | -1/+31 | |
| | | | | | | | | | | | | | | | | | | | | Otherwise assets will not work | |||||
| * | | | | Merge pull request #22600 from javan/update-action-cable | David Heinemeier Hansson | 2015-12-15 | 2 | -4/+15 | |
| |\ \ \ \ | | | | | | | | | | | | | Bring latest changes over from actioncable/master | |||||
| | * | | | | Bring latest changes over from actioncable/master | Javan Makhmali | 2015-12-15 | 2 | -4/+15 | |
| |/ / / / | ||||||
| * | | | | Switch from engine to railtie | David Heinemeier Hansson | 2015-12-15 | 2 | -6/+6 | |
| | | | | | ||||||
| * | | | | Use a bind param for `LIMIT` and `OFFSET` | Sean Griffin | 2015-12-15 | 5 | -13/+47 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We currently generate an unbounded number of prepared statements when `limit` or `offset` are called with a dynamic argument. This changes `LIMIT` and `OFFSET` to use bind params, eliminating the problem. `Type::Value#hash` needed to be implemented, as it turns out we busted the query cache if the type object used wasn't exactly the same object. This drops support for passing an `Arel::Nodes::SqlLiteral` to `limit`. Doing this relied on AR internals, and was never officially supported usage. Fixes #22250. | |||||
| * | | | | Deprecate limit strings with commas | Sean Griffin | 2015-12-15 | 2 | -3/+13 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some backends allow `LIMIT 1,2` as a shorthand for `LIMIT 1 OFFSET 2`. Supporting this in Active Record massively complicates using bind parameters for limit and offset, and it's trivially easy to build an invalid SQL query by also calling `offset` on the same `Relation`. This is a niche syntax that is only supported by a few adapters, and can be trivially worked around by calling offset explicitly. | |||||
| * | | | | Add specific bug reporting guidelines to contributing.md. [ci skip] | Tim | 2015-12-15 | 1 | -7/+33 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add boldface and bullets to make more readable. Make sentences declarative. Remove "you"s. | |||||
| * | | | | Add option to skip Action Cable | Rafael Mendonça França | 2015-12-14 | 3 | -2/+10 | |
| | | | | | ||||||
| * | | | | Do not run action cable tests in isolation | Rafael Mendonça França | 2015-12-14 | 1 | -0/+1 | |
| | | | | | | | | | | | | | | | | | | | | We don't have isolation tests for it yet. | |||||
| * | | | | Hook in ActionView::Base, not in ApplicationController | Rafael Mendonça França | 2015-12-14 | 1 | -2/+4 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The application can have more than one ApplicationController and we want Action Cable helpers in all controllers and their views so we should hook in the same place that others helpers. | |||||
| * | | | | Require the top level file in the engine | Rafael Mendonça França | 2015-12-14 | 1 | -0/+1 | |
| | | | | | ||||||
| * | | | | Add Action Cable tests to travis matrix | Rafael Mendonça França | 2015-12-14 | 2 | -0/+2 | |
| | | | | | ||||||
| * | | | | Update Gemfile.lock | Rafael Mendonça França | 2015-12-14 | 1 | -5/+0 | |
| | | | | | ||||||
| * | | | | Add action_cable to rails/all and application.rb | Rafael Mendonça França | 2015-12-14 | 2 | -9/+11 | |
| | | | | | ||||||
| * | | | | Remove action cable gems that are already in the gemspec | Rafael Mendonça França | 2015-12-14 | 1 | -6/+0 | |
| | | | | | ||||||
| * | | | | Remove Alpha disclaimer from ActionCable | claudiob | 2015-12-14 | 1 | -21/+0 | |
| | | | | | | | | | | | | | | | | | | | | [ci skip] Getting to ready to merge into rails/rails | |||||
| * | | | | Initial stab at adding Action Cable to rails/master | David Heinemeier Hansson | 2015-12-14 | 64 | -1/+3547 | |
| |\ \ \ \ | ||||||
| | * | | | | Get ready to merge into Rails | David Heinemeier Hansson | 2015-12-14 | 62 | -0/+0 | |
| | | | | | | ||||||
| | * | | | | Merge pull request #137 from rails/freshen-client-side-examples | David Heinemeier Hansson | 2015-12-13 | 1 | -23/+47 | |
| | |\ \ \ \ | | | | | | | | | | | | | | | Freshen up the client-side subscription examples. Fixes #118 | |||||
| | | * | | | | Associate comments with `perform` | Javan Makhmali | 2015-12-11 | 1 | -2/+2 | |
| | | | | | | | ||||||
| | | * | | | | Freshen up the client-side subscription examples. Fixes #118 | Javan Makhmali | 2015-12-11 | 1 | -24/+48 | |
| | | | | | | | ||||||
| | * | | | | | Merge pull request #135 from gregmolnar/readme | David Heinemeier Hansson | 2015-12-13 | 1 | -0/+1 | |
| | |\ \ \ \ \ | | | | | | | | | | | | | | | | | add devise example to readme | |||||
| | | * | | | | | add devise example to readme | Greg Molnar | 2015-12-11 | 1 | -0/+1 | |
| | | | | | | | | ||||||
| | * | | | | | | Merge pull request #73 from smellsblue/websocket-url-differentiation | David Heinemeier Hansson | 2015-12-13 | 5 | -1/+60 | |
| | |\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | Easy websocket url configuration | |||||
| | | * | | | | | | Easy websocket url configuration | Mike Virata-Stone | 2015-12-11 | 5 | -1/+60 | |
| | | | |/ / / / | | | |/| | | | | | | | | | | | | | | | | | | | | Thanks to feedback from @ascrazy, @jeremy, and @javan | |||||
| | * | | | | | | Merge pull request #130 from adamliesko/allow_regexps_allowed_origins | David Heinemeier Hansson | 2015-12-13 | 3 | -4/+18 | |
| | |\ \ \ \ \ \ | | | |/ / / / / | | |/| | | | | | Allow regexp for a allowed_request_origins array | |||||
| | | * | | | | | Remove unused method allowed_origins in Connection::Base | adamliesko | 2015-12-13 | 1 | -5/+0 | |
| | | | | | | | | ||||||
| | | * | | | | | Allow regexp for a allowed_request_origins array | adamliesko | 2015-12-12 | 3 | -4/+23 | |
| | |/ / / / / | ||||||
| | * | | | | | Merge pull request #57 from lsylvester/log-to-stdout-in-developement-mode | David Heinemeier Hansson | 2015-12-11 | 2 | -0/+10 | |
| | |\ \ \ \ \ | | | | | | | | | | | | | | | | | Add logging to stdout in development mode. | |||||
| | | * | | | | | log to stdout in development mode. | Lachlan Sylvester | 2015-08-10 | 2 | -0/+10 | |
| | | | | | | | | ||||||
| | * | | | | | | Merge pull request #115 from habermann24/master | David Heinemeier Hansson | 2015-12-11 | 5 | -4/+57 | |
| | |\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | Fix an error when using multiple gid identifiers | |||||
| | | * | | | | | | Use to_gid_param for connection identifiers | Jan Habermann | 2015-11-18 | 2 | -2/+6 | |
| | | | | | | | | | ||||||
| | | * | | | | | | Add multiple identifiers test | Jan Habermann | 2015-11-06 | 4 | -2/+51 | |
| | | | | | | | | | ||||||
| | | * | | | | | | Fix an error when using multiple gid identifiers | Jan Habermann | 2015-11-05 | 1 | -1/+1 | |
| | | | | | | | | | ||||||
| | * | | | | | | | Merge pull request #125 from lsylvester/handle-custom-loggers | David Heinemeier Hansson | 2015-12-11 | 3 | -8/+14 | |
| | |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | Handle cases where logger is not a tagged logger. | |||||
| | | * | | | | | | | 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. | |||||
| | * | | | | | | | | Merge pull request #126 from mikecarroll/add_allowed_request_origins_to_readme | David Heinemeier Hansson | 2015-12-11 | 1 | -3/+24 | |
| | |\ \ \ \ \ \ \ \ | | | |_|_|_|/ / / / | | |/| | | | | | | | README.md instructions for configuring allowed request origins | |||||
| | | * | | | | | | | Add README.md instructions on configuring ↵ | Michael Carroll | 2015-11-20 | 1 | -3/+24 | |
| | | |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | ActionCable.server.config.allowed_request_origins | |||||
| | * | | | | | | | Merge pull request #133 from assimovt/master | David Heinemeier Hansson | 2015-12-09 | 1 | -1/+1 | |
| | |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | Update README find_verified_user example |