Skip to content

Commit

Permalink
add capability registry comment explaining why we do not validate nod…
Browse files Browse the repository at this point in the history
…e operator name
  • Loading branch information
aiagentdev-1990 committed Jun 5, 2024
1 parent f57ca21 commit 06b8f29
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 0 deletions.
5 changes: 5 additions & 0 deletions .changeset/famous-timers-love.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"chainlink": patch
---

#internal add capability registry comment explaining why we do not validate node operator name
5 changes: 5 additions & 0 deletions contracts/.changeset/chilly-rivers-study.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'@chainlink/contracts': patch
---

add capability registry comment explaining why we do not validate node operator name
4 changes: 4 additions & 0 deletions contracts/src/v0.8/keystone/CapabilityRegistry.sol
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,10 @@ contract CapabilityRegistry is OwnerIsCreator, TypeAndVersionInterface {
/// operator
address admin;
/// @notice Human readable name of a Node Operator managing the node
/// @dev The contract does not validate the length of the node operator
/// name and does not validate whether or not it contains invalid
/// characters. These names will be supplied by a trusted admin
/// and we want to minimize gas costs by omitting these checks on-chain.
string name;
}

Expand Down

0 comments on commit 06b8f29

Please sign in to comment.