Merge pull request #929 from graphprotocol/pcv/fix-leading-zeros-alias

fix: ensure L2 aliased addresses are the correct length
graphprotocol · Jan 25, 2024 · 726bfb2 · 726bfb2
2 parents 7b9766d + c5641c5
commit 726bfb2
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/.changeset/ten-moons-lay.md b/.changeset/ten-moons-lay.md
@@ -0,0 +1,5 @@
+---
+"@graphprotocol/sdk": patch
+---
+
+Ensure L2 aliased addresses are the correct length
diff --git a/packages/sdk/src/utils/arbitrum/address.ts b/packages/sdk/src/utils/arbitrum/address.ts
@@ -1,3 +1,4 @@
+import { hexZeroPad } from 'ethers/lib/utils'
 import { toBN } from '../units'
 
 // Adapted from:
@@ -8,5 +9,5 @@ export const applyL1ToL2Alias = (l1Address: string): string => {
   const l2AddressAsNumber = l1AddressAsNumber.add(offset)
 
   const mask = toBN(2).pow(160)
-  return l2AddressAsNumber.mod(mask).toHexString()
+  return hexZeroPad(l2AddressAsNumber.mod(mask).toHexString(), 20)
 }