search

fly-apps / dockerfile-rails

Provides a Rails generator to produce Dockerfiles and related files.
MIT License
454 stars 38 forks source link

The app scanner for ActionCable should check if the config needs Redis #96

Open albus522 opened 1 month ago

albus522 commented 1 month ago

Right now, if ActionCable is being using, we try to add redis even if ActionCable is not configured to use redis. It would be better to make the check a bit smarter in determining if redis is actually required.

rubys commented 1 month ago

Take a peek at https://github.com/fly-apps/dockerfile-rails/blob/main/lib/dockerfile-rails/scanner.rb, and look for references to anycable.

Also, you can experiment with:

bin/rails console
> YAML.load_file('config/cable.yml')