Skip to content

Commit

Permalink
Finish migration from int64 durations to durationpb (#7147)
Browse files Browse the repository at this point in the history
This is a cleanup PR finishing the migration from int64 durations to
protobuf `*durationpb.Duration` by removing all usage of the old int64
fields. In the previous PR
#7146 all fields were
switched to read from the protobuf durationpb fields.

Fixes #7097
  • Loading branch information
pgporada authored Nov 28, 2023
1 parent 6925fad commit 51e9f39
Show file tree
Hide file tree
Showing 4 changed files with 566 additions and 583 deletions.
5 changes: 2 additions & 3 deletions ra/ra.go
Original file line number Diff line number Diff line change
Expand Up @@ -1496,9 +1496,8 @@ func (ra *RegistrationAuthorityImpl) checkCertificatesPerFQDNSetLimit(ctx contex
}

prevIssuances, err := ra.SA.FQDNSetTimestampsForWindow(ctx, &sapb.CountFQDNSetsRequest{
Domains: names,
WindowNS: limit.Window.Duration.Nanoseconds(),
Window: durationpb.New(limit.Window.Duration),
Domains: names,
Window: durationpb.New(limit.Window.Duration),
})
if err != nil {
return fmt.Errorf("checking duplicate certificate limit for %q: %s", names, err)
Expand Down
Loading

0 comments on commit 51e9f39

Please sign in to comment.