hyperledger / fabric-ca

https://wiki.hyperledger.org/display/fabric
Apache License 2.0
441 stars 712 forks source link

Log expiry overrides #384

Closed denyeart closed 11 months ago

denyeart commented 1 year ago

If cert expiry during enrollment gets overridden due to the CA's own expiry, log a message about the override (rather than a debug message).

denyeart commented 11 months ago

@bestbeforetoday

I wonder if a warning message might be more appropriate, since it is modifying the requested NotBefore / NotAfter date? Whichever way you prefer is fine; I just wanted to raise the question.

Sorry for the delay, I only work on CA about once a month :-)

Agreed, changed to Warning. And suppressed the Warning if NotBefore is not set so that the log doesn't fill up with superfluous warnings for every request.