Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use bin/rails secret in bin/production #1030

Merged
merged 1 commit into from
May 28, 2024
Merged

Conversation

lavoiesl
Copy link
Contributor

Fix:

$ bin/production
+ export RAILS_ENV=production
+ RAILS_ENV=production
++ bin/rails app:secret
+ SECRET_KEY_BASE='Unrecognized command "app:secret" (Rails::Command::UnrecognizedCommandError)
Did you mean?  app:db:reset'

Now:

$ bin/production
+ export RAILS_ENV=production
+ RAILS_ENV=production
++ bin/rails secret
+ SECRET_KEY_BASE=aec0d54abaeba02918b23da4e2ebbd08286e0dbc43ef488c03fb819cce8e7301a372b897783855d4fc253690ede1d8ed6756649167d65f41ab07952c15a994be
+ export SECRET_KEY_BASE
+ bin/rails app:assets:precompile
+ DISABLE_DATABASE_ENVIRONMENT_CHECK=true
+ bin/rails db:setup
Created database 'db/production.sqlite3'
+ RAILS_SERVE_STATIC_FILES=true
+ bin/rails server
=> Booting Puma
=> Rails 7.1.3.3 application starting in production
=> Run `bin/rails server --help` for more startup options
Puma starting in single mode...
* Puma version: 6.4.2 (ruby 3.3.0-p0) ("The Eagle of Durango")
*  Min threads: 5
*  Max threads: 5
*  Environment: production
*          PID: 33466
* Listening on http://0.0.0.0:1030
Use Ctrl-C to stop

Most likely broke in a preview Rails upgrade, but I didn't investigate more than this.

@lavoiesl lavoiesl requested review from a team, ehasrouni and jeremiedav and removed request for a team May 28, 2024 21:20
@rafaelfranca
Copy link
Member

It broke here rails/rails@55addc7

@rafaelfranca rafaelfranca merged commit b4f2a10 into main May 28, 2024
41 checks passed
@rafaelfranca rafaelfranca deleted the seb-bin-production branch May 28, 2024 21:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants