提交 7c7a4273 编写于 作者: T Trevor Turk

Rename DummyKeyGenerator -> LegacyKeyGenerator

上级 c9597985
...@@ -428,7 +428,7 @@ class EncryptedCookieJar #:nodoc: ...@@ -428,7 +428,7 @@ class EncryptedCookieJar #:nodoc:
include ChainedCookieJars include ChainedCookieJars
def initialize(parent_jar, key_generator, options = {}) def initialize(parent_jar, key_generator, options = {})
if ActiveSupport::DummyKeyGenerator === key_generator if ActiveSupport::LegacyKeyGenerator === key_generator
raise "You didn't set config.secret_key_base, which is required for this cookie jar. " + raise "You didn't set config.secret_key_base, which is required for this cookie jar. " +
"Read the upgrade documentation to learn more about this new config option." "Read the upgrade documentation to learn more about this new config option."
end end
......
require 'abstract_unit' require 'abstract_unit'
# FIXME remove DummyKeyGenerator and this require in 4.1 # FIXME remove LegacyKeyGenerator and this require in 4.1
require 'active_support/key_generator' require 'active_support/key_generator'
class FlashTest < ActionController::TestCase class FlashTest < ActionController::TestCase
...@@ -219,7 +219,7 @@ def test_redirect_to_with_adding_flash_types ...@@ -219,7 +219,7 @@ def test_redirect_to_with_adding_flash_types
class FlashIntegrationTest < ActionDispatch::IntegrationTest class FlashIntegrationTest < ActionDispatch::IntegrationTest
SessionKey = '_myapp_session' SessionKey = '_myapp_session'
Generator = ActiveSupport::DummyKeyGenerator.new('b3c631c314c0bbca50c1b2843150fe33') Generator = ActiveSupport::LegacyKeyGenerator.new('b3c631c314c0bbca50c1b2843150fe33')
class TestController < ActionController::Base class TestController < ActionController::Base
add_flash_types :bar add_flash_types :bar
......
require 'abstract_unit' require 'abstract_unit'
# FIXME remove DummyKeyGenerator and this require in 4.1 # FIXME remove LegacyKeyGenerator and this require in 4.1
require 'active_support/key_generator' require 'active_support/key_generator'
class HttpDigestAuthenticationTest < ActionController::TestCase class HttpDigestAuthenticationTest < ActionController::TestCase
...@@ -43,7 +43,7 @@ def authenticate_with_request ...@@ -43,7 +43,7 @@ def authenticate_with_request
setup do setup do
# Used as secret in generating nonce to prevent tampering of timestamp # Used as secret in generating nonce to prevent tampering of timestamp
@secret = "4fb45da9e4ab4ddeb7580d6a35503d99" @secret = "4fb45da9e4ab4ddeb7580d6a35503d99"
@request.env["action_dispatch.key_generator"] = ActiveSupport::DummyKeyGenerator.new(@secret) @request.env["action_dispatch.key_generator"] = ActiveSupport::LegacyKeyGenerator.new(@secret)
end end
teardown do teardown do
......
...@@ -7,7 +7,7 @@ ...@@ -7,7 +7,7 @@
$stderr.puts "Skipping KeyGenerator test: broken OpenSSL install" $stderr.puts "Skipping KeyGenerator test: broken OpenSSL install"
else else
# FIXME remove DummyKeyGenerator and this require in 4.1 # FIXME remove LegacyKeyGenerator and this require in 4.1
require 'active_support/key_generator' require 'active_support/key_generator'
require 'active_support/message_verifier' require 'active_support/message_verifier'
...@@ -413,29 +413,29 @@ def test_tampered_cookies ...@@ -413,29 +413,29 @@ def test_tampered_cookies
def test_raises_argument_error_if_missing_secret def test_raises_argument_error_if_missing_secret
assert_raise(ArgumentError, nil.inspect) { assert_raise(ArgumentError, nil.inspect) {
@request.env["action_dispatch.key_generator"] = ActiveSupport::DummyKeyGenerator.new(nil) @request.env["action_dispatch.key_generator"] = ActiveSupport::LegacyKeyGenerator.new(nil)
get :set_signed_cookie get :set_signed_cookie
} }
assert_raise(ArgumentError, ''.inspect) { assert_raise(ArgumentError, ''.inspect) {
@request.env["action_dispatch.key_generator"] = ActiveSupport::DummyKeyGenerator.new("") @request.env["action_dispatch.key_generator"] = ActiveSupport::LegacyKeyGenerator.new("")
get :set_signed_cookie get :set_signed_cookie
} }
end end
def test_raises_argument_error_if_secret_is_probably_insecure def test_raises_argument_error_if_secret_is_probably_insecure
assert_raise(ArgumentError, "password".inspect) { assert_raise(ArgumentError, "password".inspect) {
@request.env["action_dispatch.key_generator"] = ActiveSupport::DummyKeyGenerator.new("password") @request.env["action_dispatch.key_generator"] = ActiveSupport::LegacyKeyGenerator.new("password")
get :set_signed_cookie get :set_signed_cookie
} }
assert_raise(ArgumentError, "secret".inspect) { assert_raise(ArgumentError, "secret".inspect) {
@request.env["action_dispatch.key_generator"] = ActiveSupport::DummyKeyGenerator.new("secret") @request.env["action_dispatch.key_generator"] = ActiveSupport::LegacyKeyGenerator.new("secret")
get :set_signed_cookie get :set_signed_cookie
} }
assert_raise(ArgumentError, "12345678901234567890123456789".inspect) { assert_raise(ArgumentError, "12345678901234567890123456789".inspect) {
@request.env["action_dispatch.key_generator"] = ActiveSupport::DummyKeyGenerator.new("12345678901234567890123456789") @request.env["action_dispatch.key_generator"] = ActiveSupport::LegacyKeyGenerator.new("12345678901234567890123456789")
get :set_signed_cookie get :set_signed_cookie
} }
end end
......
require 'abstract_unit' require 'abstract_unit'
require 'stringio' require 'stringio'
# FIXME remove DummyKeyGenerator and this require in 4.1 # FIXME remove LegacyKeyGenerator and this require in 4.1
require 'active_support/key_generator' require 'active_support/key_generator'
class CookieStoreTest < ActionDispatch::IntegrationTest class CookieStoreTest < ActionDispatch::IntegrationTest
SessionKey = '_myapp_session' SessionKey = '_myapp_session'
SessionSecret = 'b3c631c314c0bbca50c1b2843150fe33' SessionSecret = 'b3c631c314c0bbca50c1b2843150fe33'
Generator = ActiveSupport::DummyKeyGenerator.new(SessionSecret) Generator = ActiveSupport::LegacyKeyGenerator.new(SessionSecret)
Verifier = ActiveSupport::MessageVerifier.new(SessionSecret, :digest => 'SHA1') Verifier = ActiveSupport::MessageVerifier.new(SessionSecret, :digest => 'SHA1')
SignedBar = Verifier.generate(:foo => "bar", :session_id => SecureRandom.hex(16)) SignedBar = Verifier.generate(:foo => "bar", :session_id => SecureRandom.hex(16))
......
...@@ -39,7 +39,7 @@ def generate_key(salt, key_size=64) ...@@ -39,7 +39,7 @@ def generate_key(salt, key_size=64)
end end
end end
class DummyKeyGenerator # :nodoc: class LegacyKeyGenerator # :nodoc:
SECRET_MIN_LENGTH = 30 # Characters SECRET_MIN_LENGTH = 30 # Characters
def initialize(secret) def initialize(secret)
......
require 'fileutils' require 'fileutils'
require 'active_support/core_ext/object/blank' require 'active_support/core_ext/object/blank'
# FIXME remove DummyKeyGenerator and this require in 4.1 # FIXME remove LegacyKeyGenerator and this require in 4.1
require 'active_support/key_generator' require 'active_support/key_generator'
require 'rails/engine' require 'rails/engine'
...@@ -112,7 +112,7 @@ def key_generator ...@@ -112,7 +112,7 @@ def key_generator
key_generator = ActiveSupport::KeyGenerator.new(config.secret_key_base, iterations: 1000) key_generator = ActiveSupport::KeyGenerator.new(config.secret_key_base, iterations: 1000)
ActiveSupport::CachingKeyGenerator.new(key_generator) ActiveSupport::CachingKeyGenerator.new(key_generator)
else else
ActiveSupport::DummyKeyGenerator.new(config.secret_token) ActiveSupport::LegacyKeyGenerator.new(config.secret_token)
end end
end end
end end
......
require 'isolation/abstract_unit' require 'isolation/abstract_unit'
# FIXME remove DummyKeyGenerator and this require in 4.1 # FIXME remove LegacyKeyGenerator and this require in 4.1
require 'active_support/key_generator' require 'active_support/key_generator'
module ApplicationTests module ApplicationTests
...@@ -10,7 +10,7 @@ def remote_ip(env = {}) ...@@ -10,7 +10,7 @@ def remote_ip(env = {})
remote_ip = nil remote_ip = nil
env = Rack::MockRequest.env_for("/").merge(env).merge!( env = Rack::MockRequest.env_for("/").merge(env).merge!(
'action_dispatch.show_exceptions' => false, 'action_dispatch.show_exceptions' => false,
'action_dispatch.key_generator' => ActiveSupport::DummyKeyGenerator.new('b3c631c314c0bbca50c1b2843150fe33') 'action_dispatch.key_generator' => ActiveSupport::LegacyKeyGenerator.new('b3c631c314c0bbca50c1b2843150fe33')
) )
endpoint = Proc.new do |e| endpoint = Proc.new do |e|
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册