cathery / sys-ftpd

Nintendo Switch FTP server as a background service (sysmodule)
GNU General Public License v3.0
240 stars 14 forks source link

Not show All SD Root folders when Bootprotect is enabled on Hekate #19

Closed impeeza closed 3 months ago

impeeza commented 3 years ago

I am using FW 13.0.0, Atmosphère 1.1.1 and Hekate 5.6.2.

If the key bootprotect=1 is enabled on hekate_ipl.ini is supposed to hide the bootloader folder on the SD root.

But FTPD appears to stop listing folders when do not have access to that folder, so the SD root on a FTP client looks like

image

The strange is what others homebrew does show the complete folder structure on SD root, by example, NX-Shell shows all the folders:

image

is like the code listing SD content do not catch exceptions so when error abort the listing processes.

xXxQuiquexXx commented 2 years ago

Same problem here! Very well detailed the problem, I add that it only happens with firmware 13 and bootprotect enabled

ToxiClay commented 2 years ago

OK, the fix for the bootprotect issue shouldn't be too intensive, as mtheall was able to produce a fixed version of FTPD in short order; hopefully, this means that it shouldn't be too hard to adapt sysmodule versions so that users can keep bootprotect enabled for safety.

Here's the commit in question: https://github.com/mtheall/ftpd/commit/9f31cde4ee40d55eb12f6ec9de30a828120dc3a3

darkarrow commented 2 years ago

Hello, just wanted to interject and say that I am having this issue as well.

Seems the ftp server doesn't know how to handle listing/not listing something that doesn't allow reading/writing and that's what the issue is.

I do get it to list shortly after setting bootprotect in hekate to on but then sometime latter it breaks and won't list the root of the sd card unless I disable bootprotect. Once I connect even though it won't let me list the root directory I can still go to different directories as long as I know the folder I want to switch to.

Now that I found an issue open for what I am having troubles with wanted to see if there is any time frame or intention to fixing this issue? Thanks.

impeeza commented 3 months ago

Thanks closing the issue.