-
Notifications
You must be signed in to change notification settings - Fork 59
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
Problem associated with stripping the last /
in the URL.
#142
Comments
The segfault is due to HTTP 404, rather than having "." in the file name, I think. I cannot reproduce locally. |
Ok, maybe you are right, can you please try on this folder? https://myrient.erista.me/files/No-Intro/Tiger%20-%20Game.com/ I mount just https://myrient.erista.me/files and then want to browse No-Intro folder |
Sure, I will give it a go later today. :) |
Hmm it hangs on my machine. Have you got a full log? When I manually kill it, it stops at |
Okay, I think the hanging might be due to some sort of rate limiting mechanism on the server. I cannot reproduce the problem you described. Which distro are you using? |
Do you own that server, btw? |
I am using Kubuntu 24.04. I do not own that server. Can attach full log later, not at the computer now. So it hangs on your system too on segfault? |
For that particular server, you could consider using https://wiki.archlinux.org/title/CurlFtpFS |
It hangs, but it doesn't segfault. |
Just some more info. It causes segfault when I access it from Krusader.
When I access it in Midnight Commander or in command line, it looks like the folder is empty.
this is the last log entry when opening in MC:
Warning:src/link.c:893:Link_download_full: cannot retrieve URL: https://myrient.erista.me/files/No-Intro/Tiger%20-%20Game.com, HTTP 404
Warning:src/link.c:893:Link_download_full: cannot retrieve URL: https://myrient.erista.me/files/No-Intro/Tiger%20-%20Game.com, HTTP 404
|
Ahh it appears to be related to the behaviour of stripping the last Try and access these two links below: The hanging is related to accessing the link without the |
/
.
/
. /
in the URL.
It indeed works weird. I cannot open that problematic folder without / on the end. But all other folders add that slash automatically and do not have that problem. Maybe the problem is that some folders ending with ".com" remove that slash? I dont understand that. |
This is related to PR #121 |
@WizzardSK , well that web server you linked to is not "well-behaved", I can't quite replicate its behaviour on my local Apache, that server you pointed at crashes my httpdirfs every time. On your system, what is its last |
It depends of which folder I check the last. Then when I switch to that bad fodler, I see this: Warning:src/link.c:893:Link_download_full: cannot retrieve URL: https://myrient.erista.me/files/No-Intro/Tiger%20-%20Game.com, HTTP 404 |
Could you post a full log please? |
Ok, I tried that in Krusader this time to show how it crashes on segfault |
Cheers! Btw, does the software crash on other servers that you use? |
Cannot confirm, cause I only use this server.
Dňa ne 12. 5. 2024, 20:11 Fufu Fang ***@***.***> napísal(a):
… Cheers!
Btw, does the software crash on other servers that you use?
—
Reply to this email directly, view it on GitHub
<#142 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AKHCB4THZRQC622JPA4E3FTZB6WENAVCNFSM6AAAAABHQUSBZ2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBWGMZTGMZXGU>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
This issue had been modified as the root cause became more clear.
It appears that stripping the last
/
can cause problems for some servers:#142 (comment)
Perhaps we need to add
/
at the end of the URL for directories.From @WizzardSK
Original text:
When I browse mounted folder which contains folder with dot in the name, it causes segfault:
Warning:src/link.c:893:Link_download_full: cannot retrieve URL: https://myserver/Tiger%20-%20Game.com, HTTP 404
./httpdirfs: line 11: 5282 Segmentation fault (core dumped) "$tmp_dir/ld-linux-x86-64.so.2" --library-path "$tmp_dir" "$tmp_dir/httpdirfs" "$@"
The text was updated successfully, but these errors were encountered: