Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: add security contact to various contract NatSpecs (OZ N-03) #1028

Merged
merged 8 commits into from
Oct 1, 2024
2 changes: 2 additions & 0 deletions packages/horizon/contracts/payments/GraphPayments.sol
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,8 @@ import { GraphDirectory } from "../utilities/GraphDirectory.sol";
* @notice This contract is part of the Graph Horizon payments protocol. It's designed
* to pull funds (GRT) from the {PaymentsEscrow} and distribute them according to a
* set of pre established rules.
* @custom:security-contact Please email [email protected] if you find any
* bugs. We may have an active bug bounty program.
*/
contract GraphPayments is Initializable, MulticallUpgradeable, GraphDirectory, IGraphPayments {
using TokenUtils for IGraphToken;
Expand Down
2 changes: 2 additions & 0 deletions packages/horizon/contracts/payments/PaymentsEscrow.sol
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,8 @@ import { GraphDirectory } from "../utilities/GraphDirectory.sol";
* @notice This contract is part of the Graph Horizon payments protocol. It holds the funds (GRT)
* for payments made through the payments protocol for services provided
* via a Graph Horizon data service.
* @custom:security-contact Please email [email protected] if you find any
* bugs. We may have an active bug bounty program.
*/
contract PaymentsEscrow is Initializable, MulticallUpgradeable, GraphDirectory, IPaymentsEscrow {
using TokenUtils for IGraphToken;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,8 @@ import { ECDSA } from "@openzeppelin/contracts/utils/cryptography/ECDSA.sol";
* @dev Note that the contract expects the RAV aggregate value to be monotonically increasing, each successive RAV for the same
* (data service-payer-receiver) tuple should have a value greater than the previous one. The contract will keep track of the tokens
* already collected and calculate the difference to collect.
* @custom:security-contact Please email [email protected] if you find any
* bugs. We may have an active bug bounty program.
*/
contract TAPCollector is EIP712, GraphDirectory, ITAPCollector {
using PPMMath for uint256;
Expand Down
2 changes: 2 additions & 0 deletions packages/horizon/contracts/staking/HorizonStaking.sol
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,8 @@ import { HorizonStakingBase } from "./HorizonStakingBase.sol";
* This is due to the contract size limit on Arbitrum (24kB). The extension contract implements functionality to support
* the legacy staking functions and the transfer tools. Both can be eventually removed without affecting the main
* staking contract.
* @custom:security-contact Please email [email protected] if you find any
* bugs. We may have an active bug bounty program.
*/
contract HorizonStaking is HorizonStakingBase, IHorizonStakingMain {
using TokenUtils for IGraphToken;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,8 @@ import { HorizonStakingBase } from "./HorizonStakingBase.sol";
* without losing rewards or having service interruptions.
* @dev TODO: Once the transition period and the transfer tools are deemed not necessary this contract
* can be removed. It's expected the transition period to last for a full allocation cycle (28 epochs).
* @custom:security-contact Please email [email protected] if you find any
* bugs. We may have an active bug bounty program.
*/
contract HorizonStakingExtension is HorizonStakingBase, IL2StakingBase, IHorizonStakingExtension {
using TokenUtils for IGraphToken;
Expand Down
4 changes: 3 additions & 1 deletion packages/subgraph-service/contracts/DisputeManager.sol
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ import { GraphDirectory } from "@graphprotocol/horizon/contracts/utilities/Graph
import { DisputeManagerV1Storage } from "./DisputeManagerStorage.sol";
import { AttestationManager } from "./utilities/AttestationManager.sol";

/*
/**
* @title DisputeManager
* @notice Provides a way to permissionlessly create disputes for incorrect behavior in the Subgraph Service.
*
Expand All @@ -40,6 +40,8 @@ import { AttestationManager } from "./utilities/AttestationManager.sol";
* Arbitration:
* Disputes can only be accepted, rejected or drawn by the arbitrator role that can be delegated
* to a EOA or DAO.
* @custom:security-contact Please email [email protected] if you find any
* bugs. We may have an active bug bounty program.
*/
contract DisputeManager is
Initializable,
Expand Down
5 changes: 5 additions & 0 deletions packages/subgraph-service/contracts/SubgraphService.sol
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,11 @@ import { PPMMath } from "@graphprotocol/horizon/contracts/libraries/PPMMath.sol"
import { Allocation } from "./libraries/Allocation.sol";
import { LegacyAllocation } from "./libraries/LegacyAllocation.sol";

/**
* @title SubgraphService contract
* @custom:security-contact Please email [email protected] if you find any
* bugs. We may have an active bug bounty program.
*/
contract SubgraphService is
Initializable,
OwnableUpgradeable,
Expand Down