From 8c474052ccaa415686980c055bf861defc9310cf Mon Sep 17 00:00:00 2001 From: swrup Date: Sat, 21 Feb 2026 22:11:13 +0100 Subject: [PATCH] --- src/http_management.ml | 48 +++++++++---------------- src/keys.ml | 16 +++++++++ src/mod_intf.mli | 4 +-- src/signatures.ml | 35 ++++++++---------- tools/offline_impl.ml | 82 ++++++++++++++++++++---------------------- 5 files changed, 86 insertions(+), 99 deletions(-) diff --git a/src/http_management.ml b/src/http_management.ml index 2e41ae85..a8dda624 100644 --- a/src/http_management.ml +++ b/src/http_management.ml @@ -3,29 +3,13 @@ open Api open Hash module Keys_get = struct - let mk_future_keys_response (module Keys : Keys.S) = - let* future_signkeys = Keys.future_signkeys () in - let* future_denoms = Keys.future_denominations () in - let master_pub = Keys.master_pub in - let denom_secmod_public_key = Keys.secmod_rsa_pub in - let signkey_secmod_public_key = Keys.secmod_eddsa_pub in - Ok - FutureKeysResponse. - { - future_denoms; - future_signkeys; - master_pub; - denom_secmod_public_key; - signkey_secmod_public_key; - } - let jsont = FutureKeysResponse.jsont let f req server _env = Logs.info (fun m -> m "GET /management/keys/"); - let keys = Vif.Server.device Devices.keys server in + let (module Keys : Keys.S) = Vif.Server.device Devices.keys server in let res = - let* v = mk_future_keys_response keys in + let* v = Keys.make_future_keys_response () in let s = Api.encode_exn jsont v in Ok s in @@ -51,7 +35,7 @@ module Keys_post = struct denom_hash; } in - verify Keys.master_pub master_sig r + verify master_sig r let verify_denom_sigs (module Keys : Keys.S) denom_sigs = let* fdn_l = Keys.future_denominations () in @@ -86,7 +70,7 @@ module Keys_post = struct signkey_pub= key; } in - verify Keys.master_pub master_sig r + verify master_sig r let verify_signkey_sigs (module Keys : Keys.S) signkey_sigs = let* fsk_l = Keys.future_signkeys () in @@ -137,7 +121,7 @@ module Denom_revoke = struct let verify (module Keys : Keys.S) h_denom_pub DenomRevocationSignature.{ master_sig } = let open Signatures.MasterDenominationKeyRevocation in - verify Keys.master_pub master_sig { h_denom_pub } + verify master_sig { h_denom_pub } let do_ (module Keys : Keys.S) h_denom_pub DenomRevocationSignature.{ master_sig } = @@ -163,7 +147,7 @@ module Signkey_revoke = struct let verify (module Keys : Keys.S) exchange_pub SignkeyRevocationSignature.{ master_sig } = let open Signatures.MasterSigningKeyRevocation in - verify Keys.master_pub master_sig { exchange_pub } + verify master_sig { exchange_pub } let do_ (module Keys : Keys.S) exchange_pub SignkeyRevocationSignature.{ master_sig } = @@ -196,7 +180,7 @@ module Auditors = struct validity_start; } = let open Signatures.MasterAddAuditor in - verify Keys.master_pub master_sig + verify master_sig { start_date= validity_start; auditor_pub; @@ -243,7 +227,7 @@ module Auditors_disable = struct let verify (module Keys : Keys.S) auditor_pub AuditorTeardownMessage.{ master_sig; validity_end } = let open Signatures.MasterDelAuditor in - verify Keys.master_pub master_sig { end_date= validity_end; auditor_pub } + verify master_sig { end_date= validity_end; auditor_pub } let do_ ~db_conn auditor_pub AuditorTeardownMessage.{ master_sig= _; validity_end } = @@ -291,7 +275,7 @@ module Wire_fee = struct wire_fee; } = let open Signatures.MasterWireFee in - verify Keys.master_pub master_sig_wire + verify master_sig_wire { h_wire_method= Hash.Cstring.H64.hash wire_method; start_date= fee_start; @@ -353,7 +337,7 @@ module Global_fees = struct master_sig; } = let open Signatures.GlobalFees in - verify Keys.master_pub master_sig + verify master_sig { start_date; end_date; @@ -426,7 +410,7 @@ module Wire = struct let debit_restrictions = "" in let* () = let open Signatures.MasterWireDetails in - verify Keys.master_pub master_sig_wire + verify master_sig_wire { h_wire_details= FullPaytoHash.hash payto_uri; h_conversion_url= Hash.Cstring.H64.hash conversion_url; @@ -436,7 +420,7 @@ module Wire = struct in let* () = let open Signatures.MasterAddWire in - verify Keys.master_pub master_sig_add + verify master_sig_add { start_date= validity_start; h_wire= FullPaytoHash.hash payto_uri; @@ -492,7 +476,7 @@ module Wire_disable = struct let verify (module Keys : Keys.S) WireTeardownMessage.{ payto_uri; master_sig_del; validity_end } = let open Signatures.MasterDelWire in - verify Keys.master_pub master_sig_del + verify master_sig_del { end_date= validity_end; h_wire= FullPaytoHash.hash payto_uri } let do_ ~db_conn @@ -535,7 +519,7 @@ module Drain = struct amount; } = let open Signatures.MasterDrainProfit in - verify Keys.master_pub master_sig + verify master_sig { wtid; date; @@ -576,7 +560,7 @@ module AmlOfficer = struct } = let open Signatures.MasterAmlOfficerStatus in let is_active = match is_active with true -> 1_l | false -> 0_l in - verify Keys.master_pub master_sig + verify master_sig { change_date; officer_pub; @@ -616,7 +600,7 @@ module Partners = struct wad_fee; } = let open Signatures.PartnerConfiguration in - verify Keys.master_pub master_sig + verify master_sig { partner_pub; start_date; diff --git a/src/keys.ml b/src/keys.ml index 1088f526..ecfa1ff5 100644 --- a/src/keys.ml +++ b/src/keys.ml @@ -173,6 +173,22 @@ module Make (Conn : Pg.CONN) = struct in List.filter_map Fun.id l + let make_future_keys_response () = + let* future_signkeys = future_signkeys () in + let* future_denoms = future_denominations () in + let master_pub = master_pub in + let denom_secmod_public_key = secmod_rsa_pub in + let signkey_secmod_public_key = secmod_eddsa_pub in + Ok + Api.FutureKeysResponse. + { + future_denoms; + future_signkeys; + master_pub; + denom_secmod_public_key; + signkey_secmod_public_key; + } + let sk_of_future_sk future_sk master_sig = let Api.FutureSignKey. { key; stamp_start; stamp_expire; stamp_end; signkey_secmod_sig= _ } = diff --git a/src/mod_intf.mli b/src/mod_intf.mli index c0edfacd..1bea500c 100644 --- a/src/mod_intf.mli +++ b/src/mod_intf.mli @@ -3,9 +3,6 @@ open Crypto type 'a result = ('a, string) Result.t module type KEYS = sig - val master_pub : eddsa_pub - val secmod_rsa_pub : eddsa_pub - val secmod_eddsa_pub : eddsa_pub val sign : eddsa_pub -> string -> eddsa_sig val sign_denom : rsa_pub -> string -> rsa_sig val find_signkey : eddsa_pub -> Signkey.t option result @@ -14,6 +11,7 @@ module type KEYS = sig val denominations : unit -> Denomination.t list result val future_signkeys : unit -> Api.FutureSignKey.t list result val future_denominations : unit -> Api.FutureDenom.t list result + val make_future_keys_response : unit -> Api.FutureKeysResponse.t result val certify_future_signkey : eddsa_pub -> Signatures.ExchangeSigningKeyValidity.t -> unit result diff --git a/src/signatures.ml b/src/signatures.ml index fa5c8507..2371cedc 100644 --- a/src/signatures.ml +++ b/src/signatures.ml @@ -184,14 +184,9 @@ module MK (R : R) : SIGNATURE with type r := R.r = struct end module MK_master_sig (R : R) = struct - open Crypto include MK (R) - let to_string = Bin.to_string R.bin - - let verify t r = - EddsaSignature.verify ~key:Config.Exchange.master_public_key t - ~msg:(to_string r) + let verify = verify Config.Exchange.master_public_key end (* ---- *) @@ -315,7 +310,7 @@ module DenominationKeyValidity = struct |> sealr end - include MK (R) + include MK_master_sig (R) end module ExchangeSigningKeyValidity = struct @@ -342,7 +337,7 @@ module ExchangeSigningKeyValidity = struct |> sealr end - include MK (R) + include MK_master_sig (R) end module MasterDenominationKeyRevocation = struct @@ -360,7 +355,7 @@ module MasterDenominationKeyRevocation = struct |> sealr end - include MK (R) + include MK_master_sig (R) end module MasterSigningKeyRevocation = struct @@ -378,7 +373,7 @@ module MasterSigningKeyRevocation = struct |> sealr end - include MK (R) + include MK_master_sig (R) end module MasterAddAuditor = struct @@ -402,7 +397,7 @@ module MasterAddAuditor = struct |> sealr end - include MK (R) + include MK_master_sig (R) end module MasterDelAuditor = struct @@ -423,7 +418,7 @@ module MasterDelAuditor = struct |> sealr end - include MK (R) + include MK_master_sig (R) end module GlobalFees = struct @@ -478,7 +473,7 @@ module GlobalFees = struct |> sealr end - include MK (R) + include MK_master_sig (R) end module MasterWireDetails = struct @@ -516,7 +511,7 @@ module MasterWireDetails = struct |> sealr end - include MK (R) + include MK_master_sig (R) end module MasterAddWire = struct @@ -558,7 +553,7 @@ module MasterAddWire = struct |> sealr end - include MK (R) + include MK_master_sig (R) end module MasterDelWire = struct @@ -579,7 +574,7 @@ module MasterDelWire = struct |> sealr end - include MK (R) + include MK_master_sig (R) end module MasterDrainProfit = struct @@ -607,7 +602,7 @@ module MasterDrainProfit = struct |> sealr end - include MK (R) + include MK_master_sig (R) end module MasterAmlOfficerStatus = struct @@ -633,7 +628,7 @@ module MasterAmlOfficerStatus = struct |> sealr end - include MK (R) + include MK_master_sig (R) end module PartnerConfiguration = struct @@ -672,7 +667,7 @@ module PartnerConfiguration = struct |> sealr end - include MK (R) + include MK_master_sig (R) end module WadPartnerSignature = struct @@ -748,7 +743,7 @@ module MasterWireFee = struct |> sealr end - include MK (R) + include MK_master_sig (R) end module ExchangeKeyValidity = struct diff --git a/tools/offline_impl.ml b/tools/offline_impl.ml index 47a0d22c..8364b54c 100644 --- a/tools/offline_impl.ml +++ b/tools/offline_impl.ml @@ -5,7 +5,15 @@ module Future_keys = struct open Crypto open Api - let verify = + let verify offline_master_public_key + FutureKeysResponse. + { + future_denoms; + future_signkeys; + master_pub; + denom_secmod_public_key; + signkey_secmod_public_key; + } = let verify_future_denom ~sm_denom_pub FutureDenom. { @@ -44,37 +52,35 @@ module Future_keys = struct verify sm_signkey_pub signkey_secmod_sig { exchange_pub; anchor_time; duration } in - fun our_master_public_key + let* () = + match master_pub = offline_master_public_key with + | false -> + Fmt.error + "master public key of the future key response does not match ours" + | true -> Ok () + in + let* () = + list_iter + (verify_future_denom ~sm_denom_pub:denom_secmod_public_key) + future_denoms + in + let* () = + list_iter + (verify_future_signkey ~sm_signkey_pub:signkey_secmod_public_key) + future_signkeys + in + Ok () + + let make ~master_key FutureKeysResponse. { future_denoms; future_signkeys; - master_pub; - denom_secmod_public_key; - signkey_secmod_public_key; - } - -> - let* () = - match master_pub = our_master_public_key with - | false -> - Fmt.error - "master public key of the future key response does not match ours" - | true -> Ok () - in - let* () = - list_iter - (verify_future_denom ~sm_denom_pub:denom_secmod_public_key) - future_denoms - in - let* () = - list_iter - (verify_future_signkey ~sm_signkey_pub:signkey_secmod_public_key) - future_signkeys - in - Ok () - - let make = - let denom_signature ~master_key + master_pub= _; + denom_secmod_public_key= _; + signkey_secmod_public_key= _; + } = + let denom_signature FutureDenom. { section_name= _; @@ -113,7 +119,7 @@ module Future_keys = struct in DenomSignature.{ h_denom_pub; master_sig } in - let signkey_signature ~master_key + let signkey_signature FutureSignKey. { key; stamp_start; stamp_expire; stamp_end; signkey_secmod_sig= _ } = let master_sig = @@ -129,21 +135,9 @@ module Future_keys = struct in SignKeySignature.{ key; master_sig } in - fun ~master_key - FutureKeysResponse. - { - future_denoms; - future_signkeys; - master_pub= _; - denom_secmod_public_key= _; - signkey_secmod_public_key= _; - } - -> - let denom_sigs = List.map (denom_signature ~master_key) future_denoms in - let signkey_sigs = - List.map (signkey_signature ~master_key) future_signkeys - in - MasterSignatures.{ denom_sigs; signkey_sigs } + let denom_sigs = List.map denom_signature future_denoms in + let signkey_sigs = List.map signkey_signature future_signkeys in + MasterSignatures.{ denom_sigs; signkey_sigs } end (* -- *)