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

[🐞] Script 0:0 in Safari Mobile #7140

Open
Nefcanto opened this issue Dec 7, 2024 · 2 comments
Open

[🐞] Script 0:0 in Safari Mobile #7140

Nefcanto opened this issue Dec 7, 2024 · 2 comments
Labels
STATUS-1: needs triage New issue which needs to be triaged TYPE: bug Something isn't working

Comments

@Nefcanto
Copy link

Nefcanto commented Dec 7, 2024

Which component is affected?

Qwik Runtime

Describe the bug

My website constantly crashes in Safari Mobile. We used eruda and found out that we get these errors in the console:

Error: Script error.
    at :0:0

I checked the view source and realized that do have empty <script tags. For example:

<script on-document:qcinit="q-Ct-gw6dj.js#s_Zo4wsVM0Ivk" on-document:qinit="#0" q:key="BV_1"></script>

I don't know if they are commented out or not.

Reproduction

https://blck.menutime.co/

Steps to reproduce

No response

System Info

System:
    OS: Linux 6.8 Debian GNU/Linux 12 (bookworm) 12 (bookworm)
    CPU: (8) x64 Intel(R) Core(TM) i7-4790K CPU @ 4.00GHz
    Memory: 9.41 GB / 15.50 GB
    Container: Yes
    Shell: 5.2.15 - /bin/bash
  Binaries:
    Node: 22.12.0 - /usr/local/bin/node
    Yarn: 1.22.22 - /usr/local/bin/yarn
    npm: 10.9.2 - /usr/local/bin/npm
  npmPackages:
    @builder.io/partytown: ^0.10.2 => 0.10.2 
    @builder.io/qwik: ^1.11.0 => 1.11.0 
    @builder.io/qwik-auth: ^0.2.2 => 0.2.2 
    @builder.io/qwik-city: ^1.11.0 => 1.11.0 
    typescript: ^5.6.3 => 5.7.2 
    undici: ^7.1.0 => 7.1.0 
    vite: ^5.4.11 => 5.4.11

Additional Information

No response

@Nefcanto Nefcanto added STATUS-1: needs triage New issue which needs to be triaged TYPE: bug Something isn't working labels Dec 7, 2024
@wmertens
Copy link
Member

wmertens commented Dec 7, 2024

Those script tags are intentional and should be ok.

Can't you attach the safari devtools and have it breakpoint on exceptions?

@Nefcanto
Copy link
Author

Nefcanto commented Dec 7, 2024

@wmertens, I will do that. That seems to be the last choice. Yet it seems that Safari is very strict about HTML. For example we had a problem of href without rel and Safari was the only browser complaining about it and throwing errors. Is it possible for you to fill those script tags with simple dummy JS? Something like const dummy = 0; would suffice I guess.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
STATUS-1: needs triage New issue which needs to be triaged TYPE: bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants