fix(wallet): require secure-site origin on W3mFrame postMessage - #5743
fix(wallet): require secure-site origin on W3mFrame postMessage#5743SashaMIT wants to merge 1 commit into
Conversation
Host-side frame event handlers accepted any @w3m-frame/* message with no event.origin check, so a same-page iframe could spoof login/RPC responses. Require SECURE_SITE_SDK_ORIGIN and target postMessage to it. Signed-off-by: Sasha Mitchell <sash.t.mitchell@gmail.com>
|
|
@SashaMIT is attempting to deploy a commit to the Reown Team on Vercel. A member of the Team first needs to authorize it. |
|
Thank you for your contribution! We ask that you please read and sign our CTA Document before we can accept your contribution. You can sign the CTA simply by posting a Pull Request Comment with the following text: I have read the CTA Document and I hereby sign the CTA You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot. |
Summary
W3mFrameframe-event handlers accepted any@w3m-frame/*messagewith noevent.origincheck.SECURE_SITE_ORIGIN.SECURE_SITE_SDK_ORIGINonregisterFrameEventHandler/onFrameEvent, and targetpostAppEventto that origin instead of*.Test plan
postMessagefrom a same-page iframe with@w3m-frame/GET_USER_SUCCESSis ignoredMade with Cursor