Skip to content

Conversation

aarongable
Copy link
Contributor

@aarongable aarongable commented Sep 24, 2025

Within sa.addRevokedCertificate, detect if the insert failed due to a duplicate primary key, and return berrors.AlreadyRevoked in that case. This matches the behavior of sa.RevokeCertificate when an update to the certificateStatus table modifies no rows.

This change has no immediate impact, because addRevokedCertificate is only called after RevokeCertificate's update to the certificateStatus table has succeeded: if the first operation returns AlreadyRevoked, this code is never reached. However, we plan to remove the certificateStatus table in the future, so this code will need to return AlreadyRevoked when appropriate.

Part of #8322


Warning

Do not merge before #8427
Do not merge before IN-11835 is complete

@aarongable aarongable marked this pull request as ready for review September 24, 2025 22:25
@aarongable aarongable requested a review from a team as a code owner September 24, 2025 22:25
@aarongable aarongable requested a review from jprenken September 24, 2025 22:25
jprenken
jprenken previously approved these changes Sep 24, 2025
@jprenken jprenken requested review from a team and jsha and removed request for a team September 24, 2025 23:02
@aarongable aarongable marked this pull request as ready for review October 7, 2025 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants