Safe upload / download file with rails4 carrier strip

User authentication can upload and download file, this is the main purpose of my project. I want my files to download so only user authentication can download the file. For this I also use the gem carrierwave

carrierwave / wiki "How to make a secure boot" . But when I click my download url it says "HTTP / 1.1 500 Internal Server Error"

Here is the addfiles_controller.rb file:

class AddfilesController < ApplicationController
  before_action :logged_in

  def index
    @addfiles = Addfile.all
  end

  def new
    @addfile = Addfile.new
  end

  def create
    if admin_signed_in?
      @addfile = current_admin.addfiles.build(addfile_params)
    else
      @addfile = current_user.addfiles.build(addfile_params)
    end


    if @addfile.save
      redirect_to addfiles_path
    else
      render "new"
    end
  end

  def destroy
    @addfile = Addfile.find(params[:id])
    @addfile.destroy
    redirect_to addfiles_path
  end


  def download
    path = "/#{addfile.addfile}"
    send_file path, :x_sendfile=>true
  end



  private
  def addfile_params
    params.require(:addfile).permit(:name, :attachment)
  end
end

      

config / initializers / carrierwave.rb file:

CarrierWave.configure do |config|
  # These permissions will make dir and files available only to the user running
  # the servers
  config.permissions = 0600
  config.directory_permissions = 0700
  config.storage = :file
  # This avoids uploaded files from saving to public/ and so
  # they will not be available for public (non-authenticated) downloading
  config.root = Rails.root
end

      

routes.rb file:

FileDownload::Application.routes.draw do

  match "/uploads/:id/:basename.:extension", :controller => "addfiles", :action => "download", via: :get

  resources :addfiles do
    collection  do
      get 'all_users'
    end
  end
  root "addfiles#index"
  devise_for :admins
  devise_for :users

end

      

in my views:

<%= link_to File.basename(file.attachment_url), "/uploads/#{file.id}/#{File.basename(file.attachment_url)}" %>

      

attachment_uploader.rb file

class AttachmentUploader < CarrierWave::Uploader::Base

  storage :file

  def store_dir
    "uploads/#{model.class.to_s.underscore}/#{mounted_as}/#{model.id}"
  end

  def extension_white_list
    %w(pdf doc htm html docx)
  end

end

      

Error tracing ::

Started GET "/uploads/13/ARTICLE_FINAL_.pdf" for 127.0.0.1 at 2014-09-04 14:39:53 +0600
Processing by AddfilesController#download as */*
  Parameters: {"id"=>"13", "basename"=>"ARTICLE_FINAL_", "extension"=>"pdf"}
  ←[1m←[36mUser Load (0.0ms)←[0m  ←[1mSELECT "users".* FROM "users" WHERE "users"."id" = 1 ORDER BY "users"."id" ASC LIMIT 1←[0m
Completed 500 Internal Server Error in 4ms

NameError (undefined local variable or method `addfile' for #<AddfilesController:0x46baa10>):
  app/controllers/addfiles_controller.rb:37:in `download'


  Rendered C:/RailsInstaller/Ruby1.9.3/lib/ruby/gems/1.9.1/gems/actionpack-4.0.4/lib/action_dispatch/middleware/templates/rescues/_source.erb (0.0ms)
  Rendered C:/RailsInstaller/Ruby1.9.3/lib/ruby/gems/1.9.1/gems/actionpack-4.0.4/lib/action_dispatch/middleware/templates/rescues/_trace.erb (1.0ms)
  Rendered C:/RailsInstaller/Ruby1.9.3/lib/ruby/gems/1.9.1/gems/actionpack-4.0.4/lib/action_dispatch/middleware/templates/rescues/_request_and_response.erb (1.0ms)
  Rendered C:/RailsInstaller/Ruby1.9.3/lib/ruby/gems/1.9.1/gems/actionpack-4.0.4/lib/action_dispatch/middleware/templates/rescues/diagnostics.erb within rescues/layout (47.0ms)
[2014-09-04 14:39:53] ERROR Errno::ECONNRESET: An existing connection was forcibly closed by the remote host.
        C:/RailsInstaller/Ruby1.9.3/lib/ruby/1.9.1/webrick/httpserver.rb:80:in `eof?'
        C:/RailsInstaller/Ruby1.9.3/lib/ruby/1.9.1/webrick/httpserver.rb:80:in `run'
        C:/RailsInstaller/Ruby1.9.3/lib/ruby/1.9.1/webrick/server.rb:191:in `block in start_thread'

      

What is the problem? Please give me your suggestion.

+3


source to share


2 answers


Try changing your boot method like this:

addfiles_controller.rb:



def download
      send_file '#{Rails.root}/uploads/addfile/#{file.id}'
end

      

+3


source


I was able to get this to work, similar to how you set up and follow the Carrerwave Wiki .

I am also using Pundit, so I have some permission for the context.



class RecordsController < ApplicationController

  before_action :set_record, only: :download

  def download
    authorize @record
    send_file @record.file.path 
    #where file is the name of the mount_uploader from the Record class
  end

  private

    def set_record 
      @record = Record.find(params[:id]) 
    end 
end

      

0


source







All Articles