nuke a few more smells
This commit is contained in:
parent
f336947d15
commit
513cd859ea
@ -34,8 +34,10 @@ module Apache
|
||||
#
|
||||
# r301 '/here', '/there' #=> Redirect permanent "/here" "/there"
|
||||
def r301(*opt)
|
||||
if opt.first && !opt.first.kind_of?(::String)
|
||||
raise "First parameter should be a String. Did you mean to wrap this in a rewrites block? #{opt.first}"
|
||||
if first = opt.first
|
||||
if !first.kind_of?(::String)
|
||||
raise "First parameter should be a String. Did you mean to wrap this in a rewrites block? #{first}"
|
||||
end
|
||||
end
|
||||
self << "Redirect permanent #{opt.quoteize * " "}"
|
||||
end
|
||||
|
@ -3,7 +3,7 @@ module Apache
|
||||
def enable_ssl_engine(options = {})
|
||||
self + [ '', "SSLEngine on" ]
|
||||
options.each do |key, value|
|
||||
value = quoteize(value).first
|
||||
value = value.quoteize
|
||||
case key
|
||||
when :certificate_file, :certificate_key_file
|
||||
self << "SSL#{key.apachify} #{value}"
|
||||
@ -11,7 +11,7 @@ module Apache
|
||||
self << "SSLCACertificateFile #{value}"
|
||||
end
|
||||
end
|
||||
self << ""
|
||||
blank_line!
|
||||
end
|
||||
end
|
||||
end
|
||||
|
Loading…
Reference in New Issue
Block a user