TEST_CODEOWNERS/rails
3
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Merge pull request #28266 from Stellenticket/allow_disable_server_stdout_logging

Browse Source 
rails server: Allow to explicitly specify whether to output Rails's log to stdout
This commit is contained in:
Kasper Timm Hansen 2018-07-08 20:01:45 +02:00 committed by GitHub
commit 50a9ca665d
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 21 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
railties/lib/rails/commands/server/server_command.rb
View File

@ -132,13 +132,14 @@ class ServerCommand < Base # :nodoc:
desc: "Specifies whether to perform caching in development."
class_option :restart, type: :boolean, default: nil, hide: true
class_option :early_hints, type: :boolean, default: nil, desc: "Enables HTTP/2 early hints."
class_option :log_to_stdout, type: :boolean, default: nil, optional: true,
desc: "Whether to log to stdout. Enabled by default in development when not daemonized."
def initialize(args, local_options, *)
super
@original_options = local_options - %w( --restart )
deprecate_positional_rack_server_and_rewrite_to_option(@original_options)
@log_stdout = options[:daemon].blank? && (options[:environment] || Rails.env) == "development"
end
def perform
@ -166,7 +167,7 @@ def server_options
{
user_supplied_options: user_supplied_options,
server: using,
log_stdout: @log_stdout,
log_stdout: log_to_stdout?,
Port: port,
Host: host,
DoNotReverseLookup: true,
@ -256,6 +257,12 @@ def early_hints
options[:early_hints]
end
def log_to_stdout?
options.fetch(:log_to_stdout) do
options[:daemon].blank? && environment == "development"
end
end
def pid
File.expand_path(options[:pid])
end

12
railties/test/commands/server_test.rb
View File

@ -143,10 +143,22 @@ def test_log_stdout
options = parse_arguments(args)
assert_equal true, options[:log_stdout]
args = ["-e", "development", "-d"]
options = parse_arguments(args)
assert_equal false, options[:log_stdout]
args = ["-e", "production"]
options = parse_arguments(args)
assert_equal false, options[:log_stdout]
args = ["-e", "development", "--no-log-to-stdout"]
options = parse_arguments(args)
assert_equal false, options[:log_stdout]
args = ["-e", "production", "--log-to-stdout"]
options = parse_arguments(args)
assert_equal true, options[:log_stdout]
with_rack_env "development" do
args = []
options = parse_arguments(args)