Skip to content

Commit 54fa3c7

Browse files
committed
Clarify proof validation comment
Use an explicit verb to describe why the complete proof set is validated before any key resolution begins. #968 (comment) Assisted-by: Codex:gpt-5.6-sol
1 parent d55cc8f commit 54fa3c7

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

packages/fedify/src/sig/proof.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -990,9 +990,9 @@ export async function verifyPortableObjectProof(
990990
proofs.push(proof);
991991
}
992992

993-
// Complete policy validation for the whole proof set before resolving any
994-
// keys. This prevents a later non-DID or cross-authority proof from causing
995-
// unnecessary attacker-controlled document fetches.
993+
// Validate the whole proof set before resolving any keys. A later non-DID
994+
// or cross-authority proof therefore cannot cause unnecessary
995+
// attacker-controlled document fetches.
996996
for (let proofIndex = 0; proofIndex < proofs.length; proofIndex++) {
997997
const verificationMethod = proofs[proofIndex].verificationMethodId;
998998
if (verificationMethod == null) {

0 commit comments

Comments
 (0)