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

Kiwix-serve random feature returns partly broken URLs #441

Closed
kelson42 opened this issue Jul 7, 2019 · 8 comments · Fixed by #890
Closed

Kiwix-serve random feature returns partly broken URLs #441

kelson42 opened this issue Jul 7, 2019 · 8 comments · Fixed by #890
Assignees
Milestone

Comments

@kelson42
Copy link
Collaborator

kelson42 commented Jul 7, 2019

With a ZIM with the article ?_(album_de_Nena) from WPFR. The random button returns the following URL: http://localhost:8080/wikipedia_fr_articles_2019-07/A/?_(album_de_Nena) and it should return http://localhost:8080/wikipedia_fr_articles_2019-07/A/%3F_(album_de_Nena).

@kelson42
Copy link
Collaborator Author

kelson42 commented Jul 7, 2019

@mgautierfr The problem here is quite fundamental and is related to URL escaping function, but these functions are not in cause here. The problem is that you can not apply to the whole path escaping properly, you need to apply to each part (namespace, zim url) differently... at the kiwix-serve/kiwix-lib level, you don't have this information anymore!

@kelson42
Copy link
Collaborator Author

@mgautierfr Moving the random feature in the libzim should help fixing this bug properly right?

@mgautierfr
Copy link
Member

No, the random feature in libzim will just keep a random entry in the zim file.
The generation of the url will still be in kiwix-lib side.

(I'm moving the issue in kiwix-lib repository)

@mgautierfr mgautierfr transferred this issue from kiwix/kiwix-tools Jan 13, 2021
@stale
Copy link

stale bot commented Jun 5, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be now be reviewed manually. Thank you for your contributions.

@stale stale bot added the stale label Jun 5, 2021
@kelson42 kelson42 added this to the 10.1.0 milestone Dec 7, 2021
@stale stale bot removed the stale label Dec 7, 2021
@kelson42
Copy link
Collaborator Author

kelson42 commented Feb 3, 2022

@mgautierfr I would need a full analysis of the problem/fix here.

@kelson42 kelson42 modified the milestones: 10.1.0, 10.2.0 Mar 23, 2022
@stale
Copy link

stale bot commented Jul 10, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be now be reviewed manually. Thank you for your contributions.

@stale stale bot added the stale label Jul 10, 2022
@veloman-yunkan
Copy link
Collaborator

This issue seems to be a special case of #775

@stale stale bot removed the stale label Oct 23, 2022
@kelson42
Copy link
Collaborator Author

@veloman-yunkan My guess as well.

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

Successfully merging a pull request may close this issue.

3 participants