Browse Source

Rename media to avoid exposing filename (fixes #207)

closed-social-glitch-2
Andrea Faulds 7 years ago
parent
commit
7161f91313
2 changed files with 16 additions and 2 deletions
  1. +4
    -1
      app/controllers/api/v1/media_controller.rb
  2. +12
    -1
      app/controllers/settings/profiles_controller.rb

+ 4
- 1
app/controllers/api/v1/media_controller.rb View File

@ -7,7 +7,10 @@ class Api::V1::MediaController < ApiController
respond_to :json
def create
@media = MediaAttachment.create!(account: current_user.account, file: params[:file])
file = params[:file]
# Change so Paperclip won't expose the actual filename
file.original_filename = "media" + File.extname(file.original_filename)
@media = MediaAttachment.create!(account: current_user.account, file: file)
rescue Paperclip::Errors::NotIdentifiedByImageMagickError
render json: { error: 'File type of uploaded media could not be verified' }, status: 422
rescue Paperclip::Error

+ 12
- 1
app/controllers/settings/profiles_controller.rb View File

@ -20,7 +20,18 @@ class Settings::ProfilesController < ApplicationController
private
def account_params
params.require(:account).permit(:display_name, :note, :avatar, :header, :silenced)
p = params.require(:account).permit(:display_name, :note, :avatar, :header, :silenced)
if p[:avatar]
avatar = p[:avatar]
# Change so Paperclip won't expose the actual filename
avatar.original_filename = "media" + File.extname(avatar.original_filename)
end
if p[:header]
header = p[:header]
# Change so Paperclip won't expose the actual filename
header.original_filename = "media" + File.extname(header.original_filename)
end
p
end
def set_account

Loading…
Cancel
Save