Skip to content

Commit d517b4e

Browse files
authored
chore: update snapshots and audit (#367)
* Update gas snapshot * Update axios lib * Fix yarn audit findings * Fix deployment check * Add new advisory to allowlist * Test: nitro-testnode fix * Test: add nitro-testnode branch to integration tests * Revert testing changes * Add new advisory
1 parent d13b123 commit d517b4e

File tree

4 files changed

+21
-13
lines changed

4 files changed

+21
-13
lines changed

.gas-snapshot

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ ArbitrumFoundationVestingWalletTest:testOnlyOwnerCanMigrate() (gas: 16329757)
1414
ArbitrumFoundationVestingWalletTest:testOwnerCanSetBeneficiary() (gas: 16332176)
1515
ArbitrumFoundationVestingWalletTest:testProperlyInits() (gas: 16337546)
1616
ArbitrumFoundationVestingWalletTest:testRandomAddressCantSetBeneficiary() (gas: 16329656)
17-
ArbitrumFoundationVestingWalletTest:testRelease() (gas: 16451131)
17+
ArbitrumFoundationVestingWalletTest:testRelease() (gas: 16448631)
1818
ArbitrumVestingWalletFactoryTest:testDeploy() (gas: 4589688)
1919
ArbitrumVestingWalletFactoryTest:testOnlyOwnerCanCreateWallets() (gas: 1504286)
2020
ArbitrumVestingWalletTest:testCastVote() (gas: 16201584)
@@ -26,7 +26,7 @@ ArbitrumVestingWalletTest:testDelegateFailsForNonBeneficiary() (gas: 16008435)
2626
ArbitrumVestingWalletTest:testDoesDeploy() (gas: 15971342)
2727
ArbitrumVestingWalletTest:testReleaseAffordance() (gas: 16008649)
2828
ArbitrumVestingWalletTest:testVestedAmountStart() (gas: 16074917)
29-
E2E:testE2E() (gas: 85079542)
29+
E2E:testE2E() (gas: 85074542)
3030
FixedDelegateErc20WalletTest:testInit() (gas: 5822575)
3131
FixedDelegateErc20WalletTest:testInitZeroToken() (gas: 5816805)
3232
FixedDelegateErc20WalletTest:testTransfer() (gas: 5932218)
@@ -101,7 +101,7 @@ L2SecurityCouncilMgmtFactoryTest:testRemovalGovDeployment() (gas: 30769899)
101101
L2SecurityCouncilMgmtFactoryTest:testSecurityCouncilManagerDeployment() (gas: 30788992)
102102
NomineeGovernorV2UpgradeActionTest:testAction() (gas: 8153)
103103
OfficeHoursActionTest:testConstructor() (gas: 9050)
104-
OfficeHoursActionTest:testFuzzOfficeHoursDeployment(uint256,uint256,int256,uint256,uint256,uint256) (runs: 256, μ: 317058, ~: 317184)
104+
OfficeHoursActionTest:testFuzzOfficeHoursDeployment(uint256,uint256,int256,uint256,uint256,uint256) (runs: 256, μ: 317091, ~: 317184)
105105
OfficeHoursActionTest:testInvalidConstructorParameters() (gas: 235740)
106106
OfficeHoursActionTest:testPerformBeforeMinimumTimestamp() (gas: 8646)
107107
OfficeHoursActionTest:testPerformDuringOfficeHours() (gas: 9140)
@@ -136,7 +136,7 @@ SecurityCouncilManagerTest:testRotateMember() (gas: 259133)
136136
SecurityCouncilManagerTest:testUpdateCohortAffordances() (gas: 83026)
137137
SecurityCouncilManagerTest:testUpdateFirstCohort() (gas: 295652)
138138
SecurityCouncilManagerTest:testUpdateRouter() (gas: 76296)
139-
SecurityCouncilManagerTest:testUpdateRouterAffordacnes() (gas: 112379)
139+
SecurityCouncilManagerTest:testUpdateRouterAffordacnes() (gas: 109879)
140140
SecurityCouncilManagerTest:testUpdateSecondCohort() (gas: 295657)
141141
SecurityCouncilMemberElectionGovernorTest:testCannotUseMoreVotesThanAvailable() (gas: 246997)
142142
SecurityCouncilMemberElectionGovernorTest:testCastBySig() (gas: 302852)
@@ -153,7 +153,7 @@ SecurityCouncilMemberElectionGovernorTest:testOnlyNomineeElectionGovernorCanProp
153153
SecurityCouncilMemberElectionGovernorTest:testProperInitialization() (gas: 49388)
154154
SecurityCouncilMemberElectionGovernorTest:testProposeReverts() (gas: 32916)
155155
SecurityCouncilMemberElectionGovernorTest:testRelay() (gas: 42229)
156-
SecurityCouncilMemberElectionGovernorTest:testSelectTopNominees(uint256) (runs: 256, μ: 340066, ~: 339868)
156+
SecurityCouncilMemberElectionGovernorTest:testSelectTopNominees(uint256) (runs: 256, μ: 339688, ~: 339471)
157157
SecurityCouncilMemberElectionGovernorTest:testSelectTopNomineesFails() (gas: 273335)
158158
SecurityCouncilMemberElectionGovernorTest:testSetFullWeightDuration() (gas: 34951)
159159
SecurityCouncilMemberElectionGovernorTest:testVotesToWeight() (gas: 152898)

audit-ci.jsonc

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,12 @@
4141
// form-data uses unsafe random function in form-data for choosing boundary (Critical)
4242
"GHSA-fjxv-7rqg-78g4",
4343
// tmp allows arbitrary temporary file / directory write via symbolic link `dir` parameter
44-
"GHSA-52f5-9888-hmc6"
44+
"GHSA-52f5-9888-hmc6",
45+
// web3-core-subscriptions has a Prototype Pollution vulnerability
46+
"GHSA-hhf6-3xpg-pggx",
47+
// js-yaml has prototype pollution in merge (<<)
48+
"GHSA-mh29-5h37-fv8m",
49+
// body-parser is vulnerable to denial of service when url encoding is used
50+
"GHSA-wqch-xfxh-vrr4"
4551
]
4652
}

scripts/tokenDistributorHelper.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -200,11 +200,13 @@ export async function getRecipientsDataFromContractEvents(
200200
// in 100 blocks there can be 100 recipient batches => 10k events at most
201201
const blocksToSearch = config.GET_LOGS_BLOCK_RANGE;
202202
while (currentBlock <= endBlock) {
203+
const blockTo = currentBlock + blocksToSearch > endBlock ? endBlock : currentBlock + blocksToSearch;
204+
203205
// query 100 blocks
204206
const canClaimEvents = await tokenDistributor.queryFilter(
205207
canClaimFilter,
206208
currentBlock,
207-
currentBlock + blocksToSearch
209+
blockTo
208210
);
209211

210212
// collect recipient-amount pairs

yarn.lock

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1469,9 +1469,9 @@ aws4@^1.8.0:
14691469
integrity sha512-lHe62zvbTB5eEABUVi/AwVh0ZKY9rMMDhmm+eeyuuUQbQ3+J+fONVQOZyj+DdrvD4BY33uYniyRJ4UJIaSKAfw==
14701470

14711471
axios@^1.5.1, axios@^1.6.8:
1472-
version "1.11.0"
1473-
resolved "https://registry.yarnpkg.com/axios/-/axios-1.11.0.tgz#c2ec219e35e414c025b2095e8b8280278478fdb6"
1474-
integrity sha512-1Lx3WLFQWm3ooKDYZD1eXmoGO9fxYQjrycfHFC8P0sCfQVXyROp0p9PFWBehewBOdCwHc+f/b8I0fMto5eSfwA==
1472+
version "1.13.2"
1473+
resolved "https://registry.yarnpkg.com/axios/-/axios-1.13.2.tgz#9ada120b7b5ab24509553ec3e40123521117f687"
1474+
integrity sha512-VPk9ebNqPcy5lRGuSlKx752IlDatOjT9paPlm8A7yOuW2Fbvp4X3JznJtT4f0GzGLLiWE9W8onz51SqLYwzGaA==
14751475
dependencies:
14761476
follow-redirects "^1.15.6"
14771477
form-data "^4.0.4"
@@ -4025,9 +4025,9 @@ mimic-response@^3.1.0:
40254025
integrity sha512-z0yWI+4FDrrweS8Zmt4Ej5HdJmky15+L2e6Wgn3+iK5fWzb6T3fhNFq2+MeTRb064c6Wr4N/wv0DzQTjNzHNGQ==
40264026

40274027
min-document@^2.19.0:
4028-
version "2.19.0"
4029-
resolved "https://registry.yarnpkg.com/min-document/-/min-document-2.19.0.tgz#7bd282e3f5842ed295bb748cdd9f1ffa2c824685"
4030-
integrity sha512-9Wy1B3m3f66bPPmU5hdA4DR4PB2OfDU/+GS3yAB7IQozE3tqXaVv2zOjgla7MEGSRv95+ILmOuvhLkOK6wJtCQ==
4028+
version "2.19.1"
4029+
resolved "https://registry.yarnpkg.com/min-document/-/min-document-2.19.1.tgz#7083ad4bc8879a6eba6516688e9f5d91d32e2d23"
4030+
integrity sha512-8lqe85PkqQJzIcs2iD7xW/WSxcncC3/DPVbTOafKNJDIMXwGfwXS350mH4SJslomntN2iYtFBuC0yNO3CEap6g==
40314031
dependencies:
40324032
dom-walk "^0.1.0"
40334033

0 commit comments

Comments
 (0)