Skip to content

docs(configuration): replace OP Mainnet admin addresses with Base's - #1959

Open
kriss39 wants to merge 1 commit into
base:masterfrom
kriss39:fix/configuration-base-admin-addresses
Open

docs(configuration): replace OP Mainnet admin addresses with Base's#1959
kriss39 wants to merge 1 commit into
base:masterfrom
kriss39:fix/configuration-base-admin-addresses

Conversation

@kriss39

@kriss39 kriss39 commented Sep 13, 2026

Copy link
Copy Markdown

What changed? Why?

The Admin Roles / Service Roles tables on specifications/reference/configuration list OP Mainnet's addresses, not Base's. The four linked addresses were:

Row Was Is actually
L1 ProxyAdmin Owner 0x5a0Aae59…3d2A 0x7bB41C30…595c
L2 ProxyAdmin Owner 0x6B1BAE59…4E3b (linked to optimistic.etherscan.io) 0x8cC51c30…6a6d
Challenger 0x9BA6e03D…6b3A 0x819501cd…d102
Guardian 0x09f7150D…dAf2 0x7bB41C30…595c

0x5a0Aae59… is OP Mainnet's ProxyAdmin.owner() and 0x09f7150D… is OP Mainnet's OptimismPortal.guardian(), so this looks like a leftover from copying the OP "standard configuration" page. The new values match the Contract Addresses page.

Notes to reviewers

Verified against L1 mainnet:

  • ProxyAdmin(0x0475cBCA…059E).owner()0x7bb41c3008b3f03fe483b28b8db90e19cf07595c
  • OptimismPortal(0x49048044…E97e).guardian()0x7bb41c3008b3f03fe483b28b8db90e19cf07595c
  • Base L2 ProxyAdmin(0x4200…0018).owner()0x8cc51c3008b3f03fe483b28b8db90e19cf076a6d, which is 0x7bB41C30…595c + 0x1111…1111 (aliased L1 owner, as the row says)
  • Challenger taken from the existing Base Contracts page; the previous value is OP's.

How has it been tested?

node scripts/lint-mdx.js docs/specifications/reference/configuration.mdx and node scripts/validate-docs-structure.js pass.

Screenshots

N/A (four table cells change, no layout change)

@cb-heimdall

Copy link
Copy Markdown
Collaborator

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/2
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 1
Sum 2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants