-
Notifications
You must be signed in to change notification settings - Fork 42
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
Panic on attaching iframe when getting browser window ID #1224
Comments
Hello ! Any news or Workaround for this issue ? cc. @inancgumus |
Hi @matthisholleville. This issue was reported on the community forum before here. We made some improvements. However, this issue currently exists. Can you send us the Dockerfile, scripts, and sample scripts so that we can reproduce this issue locally? This will allow us to come up with a solution faster. Thanks! |
Hi @inancgumus ! Thank you. We have identified the cause. It is due to an iframe injected by a third-party library (Stripe). Disabling this iframe in our non-production environments resolved the issue. Here is the test context that you can reuse: // scenario
await page.goto("https://app.agicap.com/fr/app/partners/cdn/connect", { waitUntil: 'load' });
await page.waitForSelector(`[data-test="${partnerSelector}"]`, { state: "visible", timeout: 20000 });
You need to run the test multiple times to see an error. It is quite flaky, but I can reproduce it frequently. |
Thanks, @matthisholleville, this is helpful 🙇 We'll consider this information while working on this issue 👍 |
@inancgumus We will probably change the way we load this iframe. In case you want to reproduce it from a local page. The iframe is loaded from this: <script src="https://js.stripe.com/v3/"></script> |
@inancgumus We have just removed the loading of this iframe when it is not necessary, and we observe a SIGNIFICANT improvement across all our scenarios. It would be interesting to understand why this particular iframe is causing issues. |
Hi @matthisholleville , I am facing a similar issue, can you share more info about how you removed/disabled the iframe? |
Hi @amirmuminovic ! First of all, in my opinion the problem is not with the iframe but with k6. We use other tools for our smokes ( pingdom ) and we've never seen this error. For our part, this iframe was loaded in the DOM of all our pages. However, it was only useful in a specific part of the application. We only load it when we try to access the specific part. It is currently IMPOSSIBLE for us to create a k6 scenario on the part that currently contains this iframe |
@matthisholleville and @amirmuminovic we believe we have fixed this issue and it has been merged into In the meantime if you wish to try it out with the fix and are comfortable with the Go ecosystem you can do so by using xk6:
Thank you for the help in resolving this issue! |
Thank you @ankur22 & @inancgumus ! |
Brief summary
Panic on attaching iframe when getting browser window ID.
xk6-browser version
v1.3.0 (browser)
OS
NA
Chrome version
NA
Docker version and image (if applicable)
NA
Steps to reproduce the problem
Test runs: 2398747, 2398780, 2688013, 2688024, 2688076, 2688087, 2842218, 2842268, 2842315
Expected behaviour
Not to Panic.
Actual behaviour
Panic.
Tasks
The text was updated successfully, but these errors were encountered: