pair() doesn't work for valid URIs in React Native / Android about walletconnect-monorepo

[linear]

CR-15 pair() doesn't work for valid URIs in React Native / Android

[ganchoradkov]

I think there is an issue with the relayer subscription of wallet connect and the event emitter is not emitting the session proposal event, It also doesnt return a status of what actually happened inside the code.

Hey @preetish-brine, could you double check that you're passing valid/unused pairing URI please. The pair method creates pairing and subscribes to the topic provided in the URI. If that topic was already subscribed by another client and proposal was consumed, you will not receive it second time.

[ganchoradkov]

Additionally, please make sure that you're subscribed to receive session_proposal before you call .pair as you could be running into race conditions due to the reliance on useEffects (judging from the snippets above)

[preetish-brine]

I think there is an issue with the relayer subscription of wallet connect and the event emitter is not emitting the session proposal event, It also doesnt return a status of what actually happened inside the code.

Hey @preetish-brine, could you double check that you're passing valid/unused pairing URI please. The pair method creates pairing and subscribes to the topic provided in the URI. If that topic was already subscribed by another client and proposal was consumed, you will not receive it second time.

I have tried to clear caches and generate new URI schemes as well , doesnt seem to work, I am already aware of the cache issue and was facing the same while integrating deep link to sign client, used to make it run by doing clear cache and generating new URI schemes

@ganchoradkov

[preetish-brine]

Additionally, please make sure that you're subscribed to receive session_proposal before you call .pair as you could be running into race conditions due to the reliance on useEffects (judging from the snippets above)
@ganchoradkov
Yes my subscription logic is same as it was for the signClient

useEffect(() => {
  if (initialized) {
    console.log("INIT_WEB3", web3WalletClient);
    web3WalletClient.on('session_proposal', onSessionProposal);
    web3WalletClient.on('session_request', onSessionRequest);
    // TODOs
    web3WalletClient.on('session_ping', (data) => console.log('ping', data));
    web3WalletClient.on('session_event', (data) => console.log('event', data));
    web3WalletClient.on('session_update', (data) => console.log('update', data));
    web3WalletClient.on('session_delete', (data) => {
      // console.log('delete', data);
      // showInfoNotification('Dapps successfully disconnected!');
    });
  }
}, [initialized, onSessionProposal, onSessionRequest]);

[ganchoradkov]

It seems that both useEffects

1. that sets the event listeners
2. the one that calls pair

are run at the same time by initialized potentially causing a race condition.
I would suggest to test the logic by having them in the same useEffect or check out our RN example wallets here https://github.com/WalletConnect/react-native-examples/tree/main/wallets

[preetish-brine]

It seems that both useEffects

1. that sets the event listeners
2. the one that calls pair

are run at the same time by initialized potentially causing a race condition. I would suggest to test the logic by having them in the same useEffect or check out our RN example wallets here https://github.com/WalletConnect/react-native-examples/tree/main/wallets

I actually tested the sign client SDK in the similar pattern and it was working in that case, i think race condition is not the actual issue,

Also i moved the pair to a callback initiated by a user button click and the behavior is same. my closest assumption of the reason of this issue is the relayer / event emitter is somehow not emitting the event

@ganchoradkov

[glitch-txs]

@preetish-brine could you provide a minimal reproducible example of the issue please?