Skip to content

Commit

Permalink
acme/autocert: return error from cachePut
Browse files Browse the repository at this point in the history
Fixes golang/go#23569

Change-Id: I0f3ffab74acd2b69da0bbec2e0e90e42c2618071
GitHub-Last-Rev: e66a888d643dacc290ef58649dcc248d1925219e
GitHub-Pull-Request: golang/crypto#35
Reviewed-on: https://go-review.googlesource.com/98756
Reviewed-by: Brad Fitzpatrick <[email protected]>
  • Loading branch information
Jake Burkhead authored and Alex Vaghin committed Mar 6, 2018
1 parent 9b4a327 commit eace95b
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion acme/autocert/renewal.go
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,9 @@ func (dr *domainRenewal) do(ctx context.Context) (time.Duration, error) {
if err != nil {
return 0, err
}
dr.m.cachePut(ctx, dr.domain, tlscert)
if err := dr.m.cachePut(ctx, dr.domain, tlscert); err != nil {
return 0, err
}
dr.m.stateMu.Lock()
defer dr.m.stateMu.Unlock()
// m.state is guaranteed to be non-nil at this point
Expand Down

0 comments on commit eace95b

Please sign in to comment.