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

Desktop: #11073: Fix directory being considered unsafe because of naming #11504

Closed
wants to merge 10 commits into from

Conversation

pedr
Copy link
Collaborator

@pedr pedr commented Dec 13, 2024

Fixes #11073

Summary

When the user tries to open a link to a folder path with dot on its name it shows the warning message about being an unknown type file.

I'm not sure if there is a security concern here, but I guess if the system recognizes the path being to a directory/folder it is probably safe to open.

@pedr pedr added bug It's a bug desktop All desktop platforms v3.2 labels Dec 13, 2024
Copy link
Collaborator

@personalizedrefrigerator personalizedrefrigerator left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for working on this!

I've left feedback on the internal Discord.

@pedr
Copy link
Collaborator Author

pedr commented Dec 16, 2024

  • Add comment on why the conditional is there

@pedr
Copy link
Collaborator Author

pedr commented Dec 16, 2024

.

@pedr
Copy link
Collaborator Author

pedr commented Dec 16, 2024

Add comment to new warning message

@laurent22
Copy link
Owner

@pedr, for this issue I've ended up adding a simple comment here: a4be891 There's no easy way to address this issue

@laurent22 laurent22 closed this Dec 18, 2024
@pedr pedr deleted the fix-folder-name-can-have-ext branch December 18, 2024 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug It's a bug desktop All desktop platforms v3.2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unknown file type warning for folders with a dot
3 participants