WalletConnect / notify-server

notify.walletconnect.com
https://docs.walletconnect.com/web3inbox/about
MIT License
20 stars 6 forks source link

fix: SDK analytics #545

Closed chris13524 closed 6 months ago

chris13524 commented 6 months ago

Description

Adds SDK version analytics as per the spec.

This adds a oneshot channel to transmit this information to outside of the handle() function which is a little strange, but seemed like a good enough solution considering the desire to have this info for both Ok and Err cases and it would be more verbose to make returning sdk in both cases compatible with ?.

This also moves the validation of the JWT payloads further up in the request processing.

Resolves #475

How Has This Been Tested?

Not tested

Due Diligence