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

Privacy plugin seems having problems with @ symbol in filename #6821

Closed
4 tasks done
dvonessen opened this issue Feb 24, 2024 · 3 comments
Closed
4 tasks done

Privacy plugin seems having problems with @ symbol in filename #6821

dvonessen opened this issue Feb 24, 2024 · 3 comments
Labels
bug Issue reports a bug resolved Issue is resolved, yet unreleased if open

Comments

@dvonessen
Copy link

Context

No response

Bug description

Hi,

I am unsure if this is a bug or not, therefore I ask here before opening an issue.
I think that the privacy plugin doesn't work with encoded @ -> %2F or any other encoded characters in filenames.

You can see the problem on gcix.dev on all the batches.
There are only two working batches docker and pypi.

Maybe I missed something in the documentation.
Repository: https://gitlab.com/gcix/gcix

Adding some logs too:

INFO    -  Documentation built in 10.61 seconds
INFO    -  [16:15:39] Reloading browsers
WARNING -  [16:15:40] "GET /assets/external/img.shields.io/gitlab/license/gcix%2Fgcix.svg HTTP/1.1" code 404
WARNING -  [16:15:40] "GET /assets/external/img.shields.io/gitlab/pipeline-status/gcix%2Fgcix.6be10e4e.svg HTTP/1.1" code 404
WARNING -  [16:15:40] "GET /assets/external/img.shields.io/gitlab/v/tag/gcix%2Fgcix.63de480b.svg HTTP/1.1" code 404
WARNING -  [16:15:40] "GET /assets/external/img.shields.io/gitlab/last-commit/Gcix%2Fgcix.d72d2dc2.svg HTTP/1.1" code 404
WARNING -  [16:15:40] "GET /assets/external/img.shields.io/gitlab/contributors/gcix%2Fgcix.63de480b.svg HTTP/1.1" code 404
INFO    -  [16:15:40] Browser connected: http://localhost:8000/
WARNING -  [16:15:40] "GET /versions.json HTTP/1.1" code 404
WARNING -  [16:15:40] "GET /assets/external/img.shields.io/npm/dm/%40gcix/gcix.37b11339.svg HTTP/1.1" code 404
WARNING -  [16:15:40] "GET /assets/external/img.shields.io/matrix/gcix%3Amatrix.4eae8735.org.svg HTTP/1.1" code 404
WARNING -  [16:15:40] "GET /assets/external/img.shields.io/gitlab/license/gcix%2Fgcix.svg HTTP/1.1" code 404
WARNING -  [16:15:40] "GET /assets/external/img.shields.io/gitlab/v/tag/gcix%2Fgcix.63de480b.svg HTTP/1.1" code 404
WARNING -  [16:15:40] "GET /assets/external/img.shields.io/gitlab/pipeline-status/gcix%2Fgcix.6be10e4e.svg HTTP/1.1" code 404

Thank you for this awesome piece of software.

Daniel

Related links

Reproduction

INFO - Archive successfully created:

9.5.11-problem-with-privacy-plugin/docs/index.md 448.0 B
9.5.11-problem-with-privacy-plugin/mkdocs.yml 103.0 B
9.5.11-problem-with-privacy-plugin/platform.json 62.0 B
9.5.11-problem-with-privacy-plugin/requirements.lock.txt 320.0 B
9.5.11-problem-with-privacy-plugin/requirements.txt 31.0 B

9.5.11-problem-with-privacy-plugin.zip 1.9 kB
9.5.11-problem-with-privacy-plugin.zip

Steps to reproduce

  1. Using batches with @ or other special characters, from shields.io
  2. Enable privacy plugin

Browser

Chrome

Before submitting

@squidfunk
Copy link
Owner

Thanks for reporting and the excellent reproduction. Fixed in ca7e952. Indeed this was a shortcoming of the privacy plugin not unquoting URLs before writing the downloaded files to the filesystem. The commit fixes this:

screenshot-localhost-3000-1708758739444

@squidfunk squidfunk added bug Issue reports a bug resolved Issue is resolved, yet unreleased if open labels Feb 24, 2024
@dvonessen
Copy link
Author

I will wait for the new release. 👍

@squidfunk
Copy link
Owner

Released as part of 9.5.12.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue reports a bug resolved Issue is resolved, yet unreleased if open
Projects
None yet
Development

No branches or pull requests

2 participants