sneak/AutistMask
1
0
Fork 0
Code Issues 50 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

chore: eth_chainId and net_version are listed in PROXY_METHODS but intercepted earlier, so both entries are dead #326

New Issue
Open
opened 2026-08-20 13:12:04 +02:00 by clawbot · 0 comments
clawbot commented 2026-08-20 13:12:04 +02:00
Collaborator
Copy Link

Found by the review of #319. Pre-existing, harmless today.

eth_chainId (src/background/index.js:626) and net_version (:642) appear in PROXY_METHODS while both are intercepted by an earlier branch in handleRpc and answered locally. The two list entries are unreachable.

No behaviour depends on them, which is why this is not in the 1.0.0 milestone. It is worth removing because the list reads as the authoritative statement of which methods are proxied to the RPC endpoint, and it currently names two that are not — so a reader auditing what leaves the extension gets a wrong answer from the obvious place to look.

Definition of done

  • Both entries removed, or the list restructured so a locally-answered method cannot appear in it.
  • Every other PROXY_METHODS entry checked against handleRpc's earlier branches for the same shadowing; report the result.
  • make check green.
Found by the review of https://git.eeqj.de/sneak/AutistMask/pulls/319. Pre-existing, harmless today. `eth_chainId` (`src/background/index.js:626`) and `net_version` (`:642`) appear in `PROXY_METHODS` while both are intercepted by an earlier branch in `handleRpc` and answered locally. The two list entries are unreachable. No behaviour depends on them, which is why this is not in the 1.0.0 milestone. It is worth removing because the list reads as the authoritative statement of which methods are proxied to the RPC endpoint, and it currently names two that are not — so a reader auditing what leaves the extension gets a wrong answer from the obvious place to look. ## Definition of done - [ ] Both entries removed, or the list restructured so a locally-answered method cannot appear in it. - [ ] Every other `PROXY_METHODS` entry checked against `handleRpc`'s earlier branches for the same shadowing; report the result. - [ ] `make check` green.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
merge-ready
needs-checks
needs-rebase
needs-review
needs-rework
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
clawbot sneak
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/AutistMask#326
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?