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

fix FileExplorer InvalidPathError #9918 #10730

Closed

Conversation

SecretLittleBoy
Copy link

@SecretLittleBoy SecretLittleBoy commented Mar 5, 2025

Description

In

      line 1531      any(sep in filename for sep in _os_alt_seps)

,the original code mistakenly treats the system's separator ('\\' or '/' ) as an illegal character, resulting in a legal path being rejected.

Closes: #9918
Closes: #9715

🎯 Target Issues

Gradio FileExplorer InvalidPathError #9918
gr.FileExplorer raises InvalidPathError() gradio.exceptions.InvalidPathError on Windows #9715

@abidlabs
Copy link
Member

abidlabs commented Mar 5, 2025

I don't think this is the fix that we want since safe_join is used in other parts of the but let me cc @aliabid94

@aliabid94
Copy link
Collaborator

@SecretLittleBoy can you see if #10735 works for you?

@abidlabs
Copy link
Member

abidlabs commented Mar 7, 2025

Closing as this is superseded by #10735

@abidlabs abidlabs closed this Mar 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants