yasirkula / UnitySimpleFileBrowser

A uGUI based runtime file browser for Unity 3D (draggable and resizable)
MIT License
849 stars 111 forks source link

UnauthorizedAccessException for system folders on Mac OS #66

Open hippogamesunity opened 2 years ago

hippogamesunity commented 2 years ago

Description of the bug

Hi! I'm testing the plugin on Mac OS and there is an issue with UnauthorizedAccessException when writing to the Root drive. The situation is:

Showing this exception to users blocks me from passing the App Store review, so I'm looking for a workarounds to hide drives or hide inaccessible locations maybe. image

Reproduction steps

Platform specs

Any Unity, any Mac

Additional info

The app asks for permissions to Documents when starting the first time.

hippogamesunity commented 2 years ago

image

yasirkula commented 2 years ago

First thing I could think of is, every time the path changes, we check for write access in that directory and if we don't have access, disable the Save button. I'll probably give it a shot sometime soon.

hippogamesunity commented 2 years ago

Thanks for your reply! Here is another screenshot from the app store reviewer team. image

yasirkula commented 2 years ago

What seems to be the issue in this screenshot?

hippogamesunity commented 2 years ago

I mean the error may occur not only for the Root drive, but for others as well.

Btw I've found code to comment to hide drives (as a workaround).

hippogamesunity commented 2 years ago

Another similar exception when listing files (not saving as before). image

hippogamesunity commented 2 years ago

All previous errors were from an old x64 Mac. On the latest Mac OS the plugin doesn't work at all due to sandbox, all pathes are incorrect (like Documents and Desktop). image

yasirkula commented 2 years ago

Thanks again for the feedback! I didn't know about sandboxing, will google it while working on a bugfix for this issue.

hippogamesunity commented 2 years ago

As you see, these paths are similar to iOS, so I can suggest Mac sandboxing is similar to iOS. BTW you can't send an app to review without adding the Sandbox capability in Xcode. At the same time, even with this feature enabled, you can't test a sandboxed app by running from Xcode. It gets really sandboxed only after downloaded from the App Store. Also I'm noob in Mac OS)

hippogamesunity commented 2 years ago

Sorry to bothering you, but I'm trying to provide as much info as I have. Thanks!

yasirkula commented 2 years ago

Note to self: useful links about app sandboxing:

Checking if directory is writeable -natively-:

Writing Mac OS native plugins:

manituan commented 1 year ago

Hi, we're facing this issue in our project and I'm wondering if you're still planning on fixing this issue. Thanks.

yasirkula commented 1 year ago

My schedule unfortunately won't allow me to fix this issue soon. I'm planning to work on this at some point but not soon.

yasirkula commented 1 month ago

@hippogamesunity @manituan If I were to add a "Browse..." button that allows the user to pick a directory using Mac's native file picker (upon completion, user will be able to browse that folder's contents using SimpleFileBrowser), do you think App Store Review team would object with the reason "if you're gonna show the native dialog anyways, then do everything with the native dialog; why show a uGUI dialog and native dialog together"?

I'd personally prefer the native dialog over uGUI dialog but there are already several native file browser plugins and some users would prefer showing their customized uGUI dialog (e.g. to display a World Space file browser). In any case, the solution will be either a fully native dialog experience or a combination of native+uGUI dialogs as I've described in my first paragraph.