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

[PreferencePage] Wrong paths are being allowed #103

Closed
jonas-jonas opened this issue Jul 16, 2019 · 0 comments · Fixed by #105
Closed

[PreferencePage] Wrong paths are being allowed #103

jonas-jonas opened this issue Jul 16, 2019 · 0 comments · Fixed by #105
Assignees
Labels
bug Something isn't working

Comments

@jonas-jonas
Copy link
Member

After #99 was merged, there are still a few paths that are detected as correct paths:

  • /
  • /usr

This is because for the check to detect these paths, bin/dart is appended to the end of the path. /bin/dart (which is / + bin/dart) evaluates to /usr/bin/dart (which is /usr + bin/dart) which is a symlink to /usr/lib/dart/bin/dart, which is technically working it's just not very pretty.

My suggestions would be to "blacklist" / and /usr as paths and always return false if these are entered. WDYT?

@jonas-jonas jonas-jonas added the bug Something isn't working label Jul 16, 2019
@jonas-jonas jonas-jonas self-assigned this Jul 16, 2019
jonas-jonas pushed a commit that referenced this issue Jul 17, 2019
jonas-jonas pushed a commit that referenced this issue Jul 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant