-
-
Notifications
You must be signed in to change notification settings - Fork 469
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
Articles containing "?" cannot be opened #2671
Comments
Hello ! |
@ibrahim1520 Yes, but first, what is your analysis of the problem? How would look like your solution? |
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. |
Hi @kelson42 , I would like to work on this issue. However I'm a newbie here so could you guide me with this issue? |
@puranjayK OK, but what isnthe root cause? How do you plan to fix the problem? |
@kelson42 I'm a newbie, so i don't have much idea what about this. Could u guide me where I need to start from and if I should even take this issue? |
@puranjayK This is not a programming lesson here. Either you have a clue how to fix it and you take it and if you have technical questions we might then help. But you need to know first what you have to do. All the necessary information should be available in the |
@kelson42 I might have come out wrong but I wasn't asking for progamming lessons. |
well I'd like to work on this issue also its funny (Varenyam Bhardwaj here) |
also im so sorry @puranjayK I totally ignored the fact that you wanted to work on this issue! |
@airpods69 could you tell me where you got the article which had this |
found it in the wikipedia (the simplified english one i forgot the name ;-;) |
Thanks @airpods69 telling me the file name. @kelson42 seems like the content doesn't exist in the kiwix library itself so it can't find it which results in not opening the next article |
you are correct but I have noticed many times that the link with the
I checked in complete version only that is https://download.kiwix.org/zim/wikipedia/wikipedia_en_simple_all_maxi_2022-01.zim |
well no thats not the issue here "whats my name?" does exist so it doesnt exist cause its the wrong link which we are going to replacing "?" with "%3F" should solve the issue while looking up for the link we are looking for: when we should be looking for: so there you go the issue should be easy to solve just need to change how the link thingy works |
okay yeah, that does solve the thing, I thought the link didn't exist. Thanks for the info ;)
I think it has to change from the library itself? |
yep that seems to be the case for rihana but lemme just check with the german one |
actually that doesnt explain one thing if it doesnt redirect then why does it work when the link is opened in a new tab |
I am not sure how libraries work but replacing "?" with "%3F" at all occurrences when loading the URL should fix it without changing everything in the library |
me and @cdhiraj40 discussed this but i thought putting it up here also would make sense so that everyone knows something interesting that i found |
Hi all, this is an issue we also had in Kiwix JS. Please see kiwix/kiwix-js#806 and the PR that fixes it kiwix/kiwix-js#807. The issue is simply that the URL needs to have any querystrings and anchors removed BEFORE it is decoded. A URL in a ZIM article is URI-encoded by default, but it may have additionally have a querystring, or an anchor (and that question mark and hash, if any, are not encoded). Usually a reader will have a function that removes any querysting and anchor before then decoding the URI and sending it to the backend for reading from the ZIM. However, if the URI is decoded first and then querystrings are removed, then of course the question mark will be interpreted as a querystring and it and everything after will be removed. You shouldn't substitute all question marks for %3F in the backend, or else you will introduce more problems than you solve! You should use a proper |
org.kiwix.kiwixmobile.core.main.CoreWebViewClient in line 143 the url "https://kiwix.app/A/What's_My_Name?" is not encoded. So that would mean that the url is decoded somewhere else? I was not able to find the place where the actual request is done. |
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. |
@Inbefortus Maybe this bug is already fixed in 3.6.0, but otherwise we will fic it in 3.7.0. |
@MohitMaliFtechiz Not a duplicate of #2847 |
@Inbefortus I will close the ticket but woukd be glad to get your feedback to confirm this is fixed in 3.7. |
@kelson42 Yes, I can confirm that this issue has been eliminated 👍 |
Kiwix Version: 3.4.4
ZIM: 2021-03 German and English Wikipedia
Articles that contain a "?" cannot be opened. You can only open them by opening them in a new tab, as demonstrated in the videos below:
Screen_Recording_20210703-220205_Kiwix.mp4
Screen_Recording_20210703-223208_Kiwix.mp4
The text was updated successfully, but these errors were encountered: