Skip to content

Conversation

@tuliomir
Copy link
Collaborator

@tuliomir tuliomir commented Nov 14, 2025

Description

Newly installed RPC Handler environments were failing to request permissions to the wallet clients, with a warning message on console informing that "requiredNamespaces are deprecated and are automatically assigned to optionalNamespaces".

This was tracked down to a change on Wallet Connect Core v2.21.0, driven by a convention from 2023. Our current staging RPC Handler uses v2.15.0, so it still has the older syntax, but any newly installed RPC server will fail to connect to the Desktop Wallet.

Acceptance Criteria

  • Adapts to a breaking change on the @walletconnect/core that makes the requiredNamespaces property obsolete

Security Checklist

  • Make sure you do not include new dependencies in the project unless strictly necessary and do not include dev-dependencies as production ones. More dependencies increase the possibility of one of them being hijacked and affecting us.

@tuliomir tuliomir requested a review from r4mmer November 14, 2025 19:37
@tuliomir tuliomir self-assigned this Nov 14, 2025
@tuliomir tuliomir added the dependencies Pull requests that update a dependency file label Nov 14, 2025
@tuliomir tuliomir moved this from Todo to In Progress (WIP) in Hathor Network Nov 14, 2025
@tuliomir tuliomir moved this from In Progress (WIP) to In Progress (Done) in Hathor Network Nov 14, 2025
@tuliomir tuliomir moved this from In Progress (Done) to In Review (WIP) in Hathor Network Nov 26, 2025
@tuliomir tuliomir merged commit a798110 into master Nov 27, 2025
2 checks passed
@github-project-automation github-project-automation bot moved this from In Review (WIP) to Waiting to be deployed in Hathor Network Nov 27, 2025
@tuliomir tuliomir deleted the fix/permission-rpcs branch November 27, 2025 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

Status: Waiting to be deployed

Development

Successfully merging this pull request may close these issues.

4 participants