From 393417efdbd7f25e8eb7624d2836f5579e7d7039 Mon Sep 17 00:00:00 2001 From: Jacob Hoffman-Andrews Date: Thu, 18 Apr 2024 14:00:53 -0700 Subject: [PATCH] sa: get order names from authorizations (#7435) This removes the only place we query the requestedNames table, which allows us to get rid of it in a subsequent PR (once this one is merged and deployed). Part of https://github.com/letsencrypt/boulder/issues/7432 --- sa/sa.go | 19 ++++++++++++++++++- 1 file changed, 18 insertions(+), 1 deletion(-) diff --git a/sa/sa.go b/sa/sa.go index a565139c77b..5eaa3163e3a 100644 --- a/sa/sa.go +++ b/sa/sa.go @@ -549,7 +549,24 @@ func (ssa *SQLStorageAuthority) NewOrderAndAuthzs(ctx context.Context, req *sapb return nil, err } - // Fourth, insert the FQDNSet entry for the order. + // Fourth, insert all of the requestedNames. + // TODO(#7432): Remove this + inserter, err = db.NewMultiInserter("requestedNames", []string{"orderID", "reversedName"}, "") + if err != nil { + return nil, err + } + for _, name := range req.NewOrder.Names { + err := inserter.Add([]interface{}{orderID, ReverseName(name)}) + if err != nil { + return nil, err + } + } + _, err = inserter.Insert(ctx, tx) + if err != nil { + return nil, err + } + + // Fifth, insert the FQDNSet entry for the order. err = addOrderFQDNSet(ctx, tx, req.NewOrder.Names, orderID, req.NewOrder.RegistrationID, req.NewOrder.Expires.AsTime()) if err != nil { return nil, err