Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Back navigation breaks communication in Safari #42

Open
Benjadahl opened this issue Oct 20, 2022 · 4 comments
Open

Back navigation breaks communication in Safari #42

Benjadahl opened this issue Oct 20, 2022 · 4 comments

Comments

@Benjadahl
Copy link

Hi,

We are using webext-bridge to create a cross platform extension deployed to Chrome and Safari on Mac and iOS.
We found an issue that occurs when using the back button, but only on Safari.
When navigating back, sometimes following sendMessage requests will never resolve. (Using the native chrome.runtime API does resolve correctly). We tried to use the [email protected] release instead - it seemed to improve the succes rate from 0% to about 50%, however it still has issues.

Would you like for us to provide a repository reproducing the issue? And is there any way else that we can contribute to fixing the issue?

Thanks a lot

@zikaari
Copy link
Collaborator

zikaari commented Oct 20, 2022

What browsers have you confirmed [email protected] to be working correctly?

@Benjadahl
Copy link
Author

Benjadahl commented Oct 20, 2022

According to our tests:

  • It works in Chrome
  • It fails in Safari (on MacOS)

@zikaari
Copy link
Collaborator

zikaari commented Oct 20, 2022

Safari is something I don't have expertise in, so unfortunately I won't be able to help code wise. Please see if you can find a solution, a PR would be very welcome.

@Benjadahl
Copy link
Author

Alright, we'll take a look tomorrow. Do you have any idea where in the code, it might be relevant for us to search?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants