Story #107192666 - bug fix. ds.alg=key.alg, move default logic to gen-digest

This commit is contained in:
Matt Farnsworth 2015-11-24 12:38:14 +02:00
parent 6c46daaa55
commit e5299d8835
2 changed files with 8 additions and 9 deletions

View file

@ -20,6 +20,7 @@ class Dnskey < ActiveRecord::Base
ALGORITHMS = Depp::Dnskey::ALGORITHMS.map {|pair| pair[1].to_s}.freeze # IANA numbers, single authority list
PROTOCOLS = %w(3)
FLAGS = %w(0 256 257) # 256 = ZSK, 257 = KSK
DS_DIGEST_TYPE = [1,2]
def epp_code_map
{
@ -67,6 +68,9 @@ class Dnskey < ActiveRecord::Base
def generate_digest
return if flags != 257 # generate ds only with KSK
self.ds_alg = alg
self.ds_digest_type = Setting.ds_algorithm if ds_digest_type.blank? || !DS_DIGEST_TYPE.include?(ds_digest_type)
flags_hex = self.class.int_to_hex(flags)
protocol_hex = self.class.int_to_hex(protocol)
alg_hex = self.class.int_to_hex(alg)
@ -74,9 +78,9 @@ class Dnskey < ActiveRecord::Base
hex = [domain.name_in_wire_format, flags_hex, protocol_hex, alg_hex, public_key_hex].join
bin = self.class.hex_to_bin(hex)
if ds_digest_type == 1
if self.ds_digest_type == 1
self.ds_digest = Digest::SHA1.hexdigest(bin).upcase
elsif ds_digest_type == 2
elsif self.ds_digest_type == 2
self.ds_digest = Digest::SHA256.hexdigest(bin).upcase
end
end

View file

@ -387,12 +387,7 @@ class Epp::Domain < Domain
end
def key_data_from(frame)
result = xm_copy frame, KEY_INTERFACE
# TODO: can these defaults go where they belong?
result.merge({
ds_alg: 3, # DSA/SHA-1 [DSA] RFC2536
ds_digest_type: Setting.ds_algorithm # only 1
})
xm_copy frame, KEY_INTERFACE
end
def ds_data_from(frame)