Skip to content

Commit 80a0ca1

Browse files
committed
Don't update peerId or peerMeta on session updates - they are not valid parameters and will always be null.
1 parent 9bbbf53 commit 80a0ca1

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

lib/src/main/kotlin/org/walletconnect/impls/WCSession.kt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -104,6 +104,8 @@ class WCSession(
104104
val requestId = createCallId()
105105
send(Session.MethodCall.SessionRequest(requestId, clientData), topic = config.handshakeTopic, callback = { resp ->
106106
(resp.result as? Map<String, *>)?.extractSessionParams()?.let { params ->
107+
peerId = params.peerData?.id
108+
peerMeta = params.peerData?.meta
107109
updateSession(params)
108110
propagateToCallbacks { onStatus(if (params.approved) Session.Status.Approved else Session.Status.Closed) }
109111
}
@@ -113,8 +115,6 @@ class WCSession(
113115
}
114116

115117
private fun updateSession(params: Session.SessionParams) {
116-
peerId = params.peerData?.id
117-
peerMeta = params.peerData?.meta
118118
approvedAccounts = params.accounts
119119
chainId = params.chainId
120120
storeSession()

0 commit comments

Comments
 (0)