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

Recommendation: Avoid Using panic! for Error Handling in file.rs #215

Open
neilnaveen opened this issue Apr 21, 2024 · 0 comments
Open

Recommendation: Avoid Using panic! for Error Handling in file.rs #215

neilnaveen opened this issue Apr 21, 2024 · 0 comments

Comments

@neilnaveen
Copy link

When handling errors in assert_is_allowed_filename in src/interface/file.rs, panic! might not be suitable. Instead, returning Result<(), String> would allow the caller to handle errors, giving more control over error management.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant