Skip to content

Commit 92a55b7

Browse files
publisher: Log serials in hex (#8844)
blog.Serial documents that its argument should be the hex-encoded form, and every other call site uses core.SerialToString. This one logged the big.Int decimal form, which matches nothing else in the system when searching or joining on serial. This PR was generated as part of an audit of #8606 using Claude Fable 5.
1 parent 6fe6193 commit 92a55b7

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

publisher/publisher.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -265,7 +265,7 @@ func (pub *Impl) SubmitToSingleCTWithResult(ctx context.Context, req *pubpb.Requ
265265
body = string(rspErr.Body)
266266
}
267267
pub.log.Info(ctx, "Failed to submit certificate to CT log",
268-
blog.Serial(cert.SerialNumber.String()),
268+
blog.Serial(core.SerialToString(cert.SerialNumber)),
269269
slog.String("issuer", cert.Issuer.CommonName),
270270
slog.String("log", ctLog.uri),
271271
slog.String("body", body),

0 commit comments

Comments
 (0)