Skip to content
This repository has been archived by the owner on Jun 21, 2023. It is now read-only.

Inject Algosigner also into Iframes #320

Open
andpay-io opened this issue Nov 2, 2021 · 6 comments
Open

Inject Algosigner also into Iframes #320

andpay-io opened this issue Nov 2, 2021 · 6 comments
Labels
enhancement New feature or request

Comments

@andpay-io
Copy link

Hi,

We're Andpay - building the first pure PSP fully on the Algorand blockchain. Lately we've integrated support with Algosigner, however we found out that Algosigner isn't compatible with our embeddable components (iframe based).

We noticed that the JS file that's being added is only injected in the top-level document frame and not on all the underlying iframes within that page. Could the Algosigner extension be updated with:

"content_scripts": [ { ... "all_frames": true, ... } ],
As explained here: https://developer.chrome.com/docs/extensions/mv3/content_scripts/#frames

This should potentially make it possible to also run Algosigner support within iframes. Hope to see this implemented soon - as currently Algosigner won't be offered as Payment option within our embeddable components.

@janmarcano janmarcano added the enhancement New feature or request label Nov 9, 2021
@andpay-io
Copy link
Author

Hi @janmarcano,

Any chance that this will be picked up in any of the upcoming releases?
Thanks!

@janmarcano
Copy link
Contributor

Hello! We're currently evaluating the impact it would have on pages with lots of iframes, as that would load AlgoSigner everywhere, even on sites that have nothing to do with AlgoSigner/Algorand.

I'll be sure to let you know if/when we decide to tackle this for a future update.

@andpay-io
Copy link
Author

Hi @janmarcano,

Thanks! Maybe another option could be to only add it to iframes that have a certain class or property. Could minimise impact, but helps re-usability for cases like ours.

@mataide
Copy link

mataide commented Apr 26, 2022

Need that as well.

@mataide
Copy link

mataide commented Apr 26, 2022

@andpay-io do you know if it's possible to do a workaround?

@andpay-io
Copy link
Author

@mataide Unfortunately not, we haven't found a workaround for this issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants