carrierwave-video-thumbnailer icon indicating copy to clipboard operation
carrierwave-video-thumbnailer copied to clipboard

Change thumbnail Content type video/mp4 to image/png

Open ranjithnalimela opened this issue 11 years ago • 23 comments

HI,

When i upload a mp4 video, video and video thumbnail both content types are storing as video/mp4. how can i set vide content type to video/mp4 and thumbnail content type image/png.

Uploder.rb:

version :thumb do
    process thumbnail: [{format: 'png', quality: 10, size: 260, workaround: true, strip: true, logger: Rails.logger}]

    def full_filename for_file
      png_name for_file, version_name
    end
end

def png_name for_file, version_name
    %Q{#{version_name}_#{for_file.chomp(File.extname(for_file))}.png}
end

ranjithnalimela avatar Jul 17 '13 07:07 ranjithnalimela

Thanx for the question. I’ll try to help you asap.

argent-smith avatar Jul 17 '13 08:07 argent-smith

Please check if the official doc help on this https://github.com/carrierwaveuploader/carrierwave#setting-the-content-type

rilian avatar Jul 17 '13 09:07 rilian

I have the same problem. My image files have a content-type of video/quicktime on amazon s3. I've added

require 'carrierwave/processing/mime_types' include CarrierWave::MimeTypes

and

process :set_content_type

but it did not fix the problem. Did anyone figure out how to solve it?

ttseng avatar Oct 22 '13 22:10 ttseng

I’ll try to, thank you.

2013/10/23 ttseng [email protected]

I have the same problem. My image files have a content-type of video/quicktime on amazon s3. I've added

require 'carrierwave/processing/mime_types' include CarrierWave::MimeTypes

and

process :set_content_type

but it did not fix the problem. Did anyone figure out how to solve it?

— Reply to this email directly or view it on GitHubhttps://github.com/evrone/carrierwave-video-thumbnailer/issues/6#issuecomment-26861570 .

Yours truly, Pavel.

argent-smith avatar Oct 23 '13 06:10 argent-smith

were you able to resolve the issue?

ttseng avatar Oct 28 '13 03:10 ttseng

Not yet. This week perhaps...

2013/10/28 ttseng [email protected]

were you able to resolve the issue?

— Reply to this email directly or view it on GitHubhttps://github.com/evrone/carrierwave-video-thumbnailer/issues/6#issuecomment-27187993 .

Yours truly, Pavel.

argent-smith avatar Oct 28 '13 07:10 argent-smith

+1

hyperrjas avatar Nov 06 '13 11:11 hyperrjas

I solved this issue by overriding the process set_content_type as follows:

require 'carrierwave/processing/mime_types'

class VideoPathUploader < CarrierWave::Uploader::Base include CarrierWave::Video include CarrierWave::Video::Thumbnailer include CarrierWave::MimeTypes

version :thumb do process thumbnail: [{format: 'png', quality: 10, size: 158, logger: Rails.logger}] def full_filename for_file png_name for_file, version_name end process :set_content_type_png # process resize_to_limit: [105, 158] end

def set_content_type_png(*args) Rails.logger.debug "#{file.content_type}" self.file.instance_variable_set(:@content_type, "image/png") end

end

I had to do this for all file types (I was using the carrierwave-video gem to convert video file types, so I had to define processes for each file type I created). It's a bit hacky, but it worked for me.

ttseng avatar Nov 17 '13 21:11 ttseng

Thanks. I’ll probably use some of it.

2013/11/18 ttseng [email protected]

I solved this issue by overriding the process set_content_type as follows:

require 'carrierwave/processing/mime_types'

class VideoPathUploader < CarrierWave::Uploader::Base include CarrierWave::Video include CarrierWave::Video::Thumbnailer include CarrierWave::MimeTypes

version :thumb do process thumbnail: [{format: 'png', quality: 10, size: 158, logger: Rails.logger}]

def full_filename for_file png_name for_file, version_name end process :set_content_type_png

process resize_to_limit: [105, 158]

end

def set_content_type_png(*args) Rails.logger.debug "#{file.content_type}" self.file.instance_variable_set(:@content_type, "video/png") end

end

I had to do this for all file types (I was using the carrierwave-video gem to convert video file types, so I had to define processes for each file type I created). It's a bit hacky, but it worked for me.

— Reply to this email directly or view it on GitHubhttps://github.com/evrone/carrierwave-video-thumbnailer/issues/6#issuecomment-28664696 .

Yours truly, Pavel.

argent-smith avatar Nov 18 '13 08:11 argent-smith

I am still facing the same issue. After using the above code I am getting the below error:

NoMethodError (undefined method `each' for "image/png":String):

Any help would be appreciated.

Thanks Rakesh

rakesh050791 avatar Sep 30 '14 07:09 rakesh050791

@ttseng would you please help clarifying @rakesh050791 ’s Q? I hope I’ll fix it in the library asap.

argent-smith avatar Sep 30 '14 08:09 argent-smith

Hello smith,

As we need the thumbnail for the video uploaded in our rails application. we are using this library for the same, but we are unable to reach at our goal.

we are using carrierwave for video uploading. but by using the code mentioned in the library for thumbnail, we are not getting the exact thumbnail for the same video.

I am attaching my video uploader screen , please see.

video_uploader

Quick reply would be appreciated.

rakesh050791 avatar Sep 30 '14 09:09 rakesh050791

we are not getting the exact thumbnail for the same video

Do you mean you see a thumbnail of a different video or what?

argent-smith avatar Sep 30 '14 09:09 argent-smith

I mean to say, i am getting the response like this

response

Please see, the url i'am getting for the video uploaded, the same url i am getting in the thumb, but this is not to be done Pavel .

Please ignore the above thumb : "" parameter, this one is my database column. made later

rakesh050791 avatar Sep 30 '14 09:09 rakesh050791

Does your version of ffmpeg/ffmpegthumbnailer support the format you use (3gp)?

argent-smith avatar Sep 30 '14 09:09 argent-smith

I have tried with mp 4 also and i've installed ffmpegthumbnailer today itself.

Could you please update me how can i check this ?

rakesh050791 avatar Sep 30 '14 09:09 rakesh050791

Just be sure it supports all the needed formats. Also, it’d be a great help if you try CLI-run of ffmpegthumbnailer on the sample file of the offending format and show me the CLI output. Thanks in advance!

argent-smith avatar Sep 30 '14 10:09 argent-smith

Just run into the same issue and @ttseng fix worked for me, are you planning to fix this anytime soon or accepting PRs for this?

00dav00 avatar Jul 08 '19 02:07 00dav00

@rakesh050791 @00dav00 thnx for the report ❤️ . Will decide ASAP.

argent-smith avatar Jul 09 '19 12:07 argent-smith

HI argent-smith , Actually i am facing with video thumbnail issue.

my issue) when i uploading a video , the thumbnail image is stretching.FOr youtube videos ,it's fine but if it's system videos,the thumbnails is stretching. I am using carrier wave gaem.Following is the my code base.

` include CarrierWave::Video # for your video processing include CarrierWave::Video::Thumbnailer

version :medium, if: :image? do process :remove_animation, if: :gif? process :resize_to_limit => [500, 500] end

version :thumb, if: :video? do process thumbnail: [{format: 'png', quality: 10, size: "500%", seek: "00:00:03", logger: Rails.logger}] def full_filename for_file png_name for_file, version_name end

def png_name for_file, version_name
  %Q{#{version_name}_#{for_file.chomp(File.extname(for_file))}.png}
end

end `

CAn you please suggest me on this?

Thank you un advance

damujay avatar Jul 06 '20 05:07 damujay

The Mime Types feature was removed from carrierwave. As of now it seems impossible to set the mime type correctly using this gem, resulting in thumbnails not displaying from on iOS.

EDIT: False alarm! You don't need to include the mime type carrierwave stuff. Just set the content type in a :process method. I just wasn't actually sending it to S3 in dev!

gregblass avatar Sep 10 '20 19:09 gregblass

Thnx for notice. I'll see what could be done with this.

argent-smith avatar Sep 14 '20 10:09 argent-smith

Helped me https://github.com/evrone/carrierwave-video-thumbnailer/issues/15#issuecomment-60081856

araslanov-e avatar Dec 30 '20 06:12 araslanov-e