Skip to content

Commit

Permalink
Remove the dead admin-revoker tool (#7941)
Browse files Browse the repository at this point in the history
The admin-revoker tool is dead. Long live the admin tool.

There's a number places that still reference admin-revoker, including
Boulder's ipki and the revocation source in the database which are still
used, even if the tool is gone. But nothing actually using the tool.
  • Loading branch information
mcpherrinm authored Jan 14, 2025
1 parent 45a56ae commit bb9d82b
Show file tree
Hide file tree
Showing 5 changed files with 1 addition and 148 deletions.
70 changes: 0 additions & 70 deletions cmd/admin-revoker/main.go

This file was deleted.

1 change: 0 additions & 1 deletion cmd/boulder/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ import (
"os"
"strings"

_ "github.com/letsencrypt/boulder/cmd/admin-revoker"
_ "github.com/letsencrypt/boulder/cmd/akamai-purger"
_ "github.com/letsencrypt/boulder/cmd/bad-key-revoker"
_ "github.com/letsencrypt/boulder/cmd/boulder-ca"
Expand Down
2 changes: 1 addition & 1 deletion ra/ra.go
Original file line number Diff line number Diff line change
Expand Up @@ -1969,7 +1969,7 @@ func (ra *RegistrationAuthorityImpl) RevokeCertByKey(ctx context.Context, req *r

// AdministrativelyRevokeCertificate terminates trust in the certificate
// provided and does not require the registration ID of the requester since this
// method is only called from the admin-revoker tool. It trusts that the admin
// method is only called from the `admin` tool. It trusts that the admin
// is doing the right thing, so if the requested reason is keyCompromise, it
// blocks the key from future issuance even though compromise has not been
// demonstrated here. It purges the certificate from the Akamai cache, and
Expand Down
38 changes: 0 additions & 38 deletions test/config-next/admin-revoker.json

This file was deleted.

38 changes: 0 additions & 38 deletions test/config/admin-revoker.json

This file was deleted.

0 comments on commit bb9d82b

Please sign in to comment.