Corrected type issue getting the evm address when requesting accounts

[tombeckenham]

Related Issue

Closes #1204

Summary of Changes

Corrected a type issue in ethRequestAccounts

Need Regression Testing

Recheck dApp connections

• Yes
• No

Risk Assessment

• Low
• Medium
• High

[github-actions]

PR Summary

Enhanced the EVM address retrieval logic in the provider controller to properly handle the EVM account type. Updated the code to correctly extract the address property from the EVM account object, adding proper type safety and validation checks. This fixes issue #1204 related to dApp connections.

Changes

File	Summary
src/background/controller/provider/controller.ts	Fixed type handling in ProviderController by properly typing evmAddress as string and correctly extracting address from evmAccount object. Added null checks and improved validation logic for EVM addresses in both initial query and approval flow.

---

autogenerated by presubmit.ai

[github-actions]

✅ LGTM!

Review Summary

Commits Considered (2)

• daf7c18: Corrected type issue getting the evm address when requesting accounts

Closes #1204

• f78a2a5: Merge pull request #1202 from onflow:1200-291

2.9.1 master

Files Processed (1)

• src/background/controller/provider/controller.ts (2 hunks)

Actionable Comments (0)

Skipped Comments (2)

• src/background/controller/provider/controller.ts [248-251]

  maintainability: "Potential duplicate error handling logic"

• src/background/controller/provider/controller.ts [238-238]

  best practice: "Variable type declaration could be more specific"