GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.
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
An uploaded file parameters will return ActionDispatch::Http::UploadedFile
That object and its tempfile should be safe to access, as they are created by Rails, and not directly from user input.
Rails Version: 5.1.4
Brakeman Version: 4.0.1
Category: File Access
Message: Parameter value used in file name
Code: FileUtils.move(params.permit(:my_upload => ([:upload])).dig("my_upload", "upload").tempfile.path, "/tmp/new_temp_file")
I agree. Will fix.
Avoid warning about file access for temp files