-
Notifications
You must be signed in to change notification settings - Fork 9.5k
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
Report not downloaded when site is accessed using IP address #5633
Comments
Also FYI, the Chrome Extension is accessed separately. Your step description indicates you're actually using v2.9 of Lighthouse built-in to DevTools. Does everything work as expected if you use v3 in Chrome Canary? |
Oh yes that's the mistake at my end I just got confused in extension and the built in dev tool. I am facing this issue every time when I am downloading the report from chrome console [you are right I am using v2.9 built in lighthouse]. I haven't check this on v3 in Chrome Canary will check and update. |
I have tried the same on v3 in Chrome Canary but again the results are same on using localhost I am able to download the report json but on changing the localhost to IP report is not downloaded. I am not sure what is going wrong in my case. :( |
Hm, it might be Windows specific then. I'm not able to reproduce on Mac. We'll have to look into it. |
@patrickhulce I fired up a VM and I can reproduce this on Windows 10 in M67/ LH2.9.1 for one specific case that I believe potentially matches the above. Windows appears to have an issue with an ip address with a host port (x.x.x.x:nnnn); this doesn't throw an error in while debugging, but it never pops the A plain no-port IP address appears to work fine, though the colon in other instances also causes odd behavior. As an example This is likely because filenames in Windows can not have colons (it's a reserved character, per docs). |
Awesome thanks so much for digging in @justinribeiro! 👍 |
BUG=GoogleChrome/lighthouse#5633 Change-Id: I58fc3d6541bfab6305936f5e5f5034b4bcfdaae1 Reviewed-on: https://chromium-review.googlesource.com/1132136 Reviewed-by: Dmitry Gozman <[email protected]> Commit-Queue: Patrick Hulce <[email protected]> Cr-Commit-Position: refs/heads/master@{#573937}
BUG=GoogleChrome/lighthouse#5633 Change-Id: I58fc3d6541bfab6305936f5e5f5034b4bcfdaae1 Reviewed-on: https://chromium-review.googlesource.com/1132136 Reviewed-by: Dmitry Gozman <[email protected]> Commit-Queue: Patrick Hulce <[email protected]> Cr-Original-Commit-Position: refs/heads/master@{#573937} Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src Cr-Mirrored-Commit: b9e6f065c035bfd0487695bd981cdf57108e5ceb
Thank you so much for your quick responses and resolutions @patrickhulce @justinribeiro :) :) |
BUG=GoogleChrome/lighthouse#5633 Change-Id: I58fc3d6541bfab6305936f5e5f5034b4bcfdaae1 Reviewed-on: https://chromium-review.googlesource.com/1132136 Reviewed-by: Dmitry Gozman <[email protected]> Commit-Queue: Patrick Hulce <[email protected]> Cr-Commit-Position: refs/heads/master@{#573937}
@swejaljain it's fixed in latest canary. |
@wardpeet thank you for the fix :) |
Bug report
Provide the steps to reproduce
P.S. If the same site is accessed on local machine with localhost using the same steps then report json is downloaded successfully.
What is the current behavior?
Report is nor downloaded on click of download report icon when web application is accessed with server IP.
What is the expected behavior?
Report should be downloaded on click of download report icon despite of the url.
Environment Information
Related issues
None
The text was updated successfully, but these errors were encountered: