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

urls alias + urls with amp #3345

Closed
inti-tui opened this issue Apr 13, 2021 · 4 comments
Closed

urls alias + urls with amp #3345

inti-tui opened this issue Apr 13, 2021 · 4 comments

Comments

@inti-tui
Copy link

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!

@soulgalore
Copy link
Member

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:
docker run --rm -v "$(pwd)":/sitespeed.io sitespeedio/sitespeed.io:16.10.3-plus1 --urlAlias search_result --n 1 --gpsi.key XXXX "https://www.sitespeed.io/?searchScope=HOTEL&departureAirports=BRN;DUS;BRU&startDate=2022-04-01&endDate=2022-04-05&showTotalPrice=0&useExtendedFilterDefaultValues=1"

@inti-tui
Copy link
Author

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?

@inti-tui
Copy link
Author

test log from your example with a real url.
test.log

@soulgalore
Copy link
Member

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

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

No branches or pull requests

2 participants