weiguoqiang/cert-manager
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

Test TestRegistry_AddClient_UpdatesClientPKChecksum must compare private key with a checksum

Browse Source 
Signed-off-by: vidarno <>
This commit is contained in:
vidarno 2023-05-03 22:17:03 +02:00
parent a1f156c2b6
commit 616a41ac8f
1 changed files with 9 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
pkg/acme/accounts/registry_test.go
View File

@ -17,6 +17,9 @@ limitations under the License.
package accounts
import (
"crypto/sha256"
"crypto/x509"
"encoding/base64"
"net/http"
"testing"
@ -157,6 +160,10 @@ func TestRegistry_AddClient_UpdatesClientPKChecksum(t *testing.T) {
t.Fatal(err)
}
pkBytes := x509.MarshalPKCS1PrivateKey(pk)
pkChecksum := sha256.Sum256(pkBytes)
pkChecksumString := base64.StdEncoding.EncodeToString(pkChecksum[:])
// Register a new client
r.AddClient(http.DefaultClient, "abc", cmacme.ACMEIssuer{}, pk, "cert-manager-test")
l := r.ListClients()
@ -164,12 +171,12 @@ func TestRegistry_AddClient_UpdatesClientPKChecksum(t *testing.T) {
t.Errorf("expected ListClients to have 1 item but it has %d", len(l))
}
isCached := r.IsKeyCheckSumCached("abc", pk)
isCached := r.IsKeyCheckSumCached(pkChecksumString, pk)
if isCached == false {
t.Fatal("checksum failed for same key")
}
isCached = r.IsKeyCheckSumCached("abc", pk2)
isCached = r.IsKeyCheckSumCached(pkChecksumString, pk2)
if isCached == true {
t.Fatal("checksum reported same for different keys")
}