-
-
Notifications
You must be signed in to change notification settings - Fork 606
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
urls alias + urls with amp #3345
Comments
Hi @inti-tui is that the exact example? The two dashes before sitespeedio/sitespeed.io:16.10.3-plus1 seems wrong. I did a quick test and this works: |
Hi, thank you for a quick feedback. The extra dashes was a copy/paste typo. No, It was not a exact example. I have tested your example and it work if you run it "manually" at the command line but not in bash script with loop for urls. The urls with quotas will expand to something like '"url"' and sitespeed lighthouse don't like it. Do you have some example with multiple urls and alias? |
test log from your example with a real url. |
Sorry for not getting back. This is the PROTOCOL_TIMEOUT error you get in Lighthouse, please checkout the Lighthouse issues if they could help: GoogleChrome/lighthouse#6512 |
Hi,
We have some urls with "&" and other reserved signs. When we trying to test them together with alias in script, always fail.
ex.
#! /bin/bash
.....
/usr/bin/docker run --rm -v /var/data/sitespeed:/sitespeed.io --sitespeedio/sitespeed.io:16.10.3-plus1 --config ./config-mobile-test.json --urlAlias search_result "https://www.mydomain.se/suchen/hotels/Mallorca/1197/list?searchScope=HOTEL&departureAirports=BRN;DUS;BRU&startDate=2022-04-01&endDate=2022-04-05&showTotalPrice=0&useExtendedFilterDefaultValues=1"
.......
Couldn't find url file at /sitespeed.io/"https:/www.mydomain.se/suchen..."
How should I do?
Very grateful if you could help me. I have test a lot with different approach but can't get it work.
Thanks!
The text was updated successfully, but these errors were encountered: