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

[Bug]: Sitemap only found 1 URL in Ecoindex-cli #62

Open
1 task done
guillaumew opened this issue Mar 1, 2024 · 4 comments
Open
1 task done

[Bug]: Sitemap only found 1 URL in Ecoindex-cli #62

guillaumew opened this issue Mar 1, 2024 · 4 comments
Labels
bug Something isn't working keep open

Comments

@guillaumew
Copy link

What happened?

Step to reproduce :
alias ecoindex-cli="docker run -it --rm --add-host=host.docker.internal:host-gateway -v /tmp/ecoindex-cli:/tmp/ecoindex-cli vvatelot/ecoindex-cli:2.26.0 ecoindex-cli"
ecoindex-cli analyze --url https://simbios.fr --sitemap https://simbios.fr/sitemap.xml --export-format json

Only 1 URL was found while the sitemap contains more than 50 elements : https://simbios.fr/sitemap.xml

Project

Ecoindex CLI

What OS do you use?

Linux

urls

https://simbios.fr/sitemap.xml

Relevant log output

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@guillaumew guillaumew added the bug Something isn't working label Mar 1, 2024
@vvatelot
Copy link
Member

vvatelot commented Mar 8, 2024

@PaulPHPE maybe you can have a look at this one ?

@PaulPHPE
Copy link
Collaborator

PaulPHPE commented Mar 8, 2024

Hello @vvatelot @guillaumew !
If you provide a url and a sitemap the cli only takes into account the url argument. You should start the analysis as follow :
ecoindex-cli analyze --sitemap https://simbios.fr/sitemap.xml --export-format json

I realize that the README should mention it. Moreover, we shouldn't ask a confirmation for sitemap scraping when we have both url and sitemap. I will make the modifications !

@guillaumew
Copy link
Author

Thanks @PaulPHPE. I didn't even realized I had this url argument. Sorry about that.

Copy link

github-actions bot commented May 8, 2024

This issue has been marked as inactive because it hasn't been updated for 30 days. If it's not updated within 7 days, it will be automatically closed. To prevent it from being closed, you can add the keep open label.

@github-actions github-actions bot added the Stale label May 8, 2024
@vvatelot vvatelot added keep open and removed Stale labels May 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working keep open
Projects
None yet
Development

No branches or pull requests

3 participants