Correct the solution to get the content type

[yopaz-giapnh]

line-bot-sdk-ruby/lib/line/bot/client.rb

Lines 1078 to 1083 in 147323c

	case file.path
	when /\.jpe?g\z/i then 'image/jpeg'
	when /\.png\z/i then 'image/png'
	else
	raise ArgumentError, "invalid file extension: #{file.path}"
	end

If using the file with no extension, this caused an error. For eg: "/var/folders/m3/p364c0313fv0mhbl3llm4bvw0000gp/T/ActiveStorage-27-20210918-49325-1bu8npn"

[kimoto]

You're right, It's too difficult to use for content_type detection.
But you can use the following method to get the content_type of the file instead of the file extension.

irb> file = open('https://www.ruby-lang.org/images/header-ruby-logo.png')
irb> file.content-type
=> "image/png"

line-bot-sdk-ruby/lib/line/bot/client.rb

Lines 1072 to 1076 in 147323c

	if file.respond_to?(:content_type)
	content_type = file.content_type
	raise ArgumentError, "invalid content type: #{content_type}" unless ['image/jpeg', 'image/png'].include?(content_type)
	content_type

But this use case is required use open-uri :(
https://ruby-doc.org/stdlib-2.5.1/libdoc/open-uri/rdoc/OpenURI/Meta.html#method-i-content_type

In my opinion, I think it would be good to be able to pass the content_type as an argument.

[Yang-33]

currently we don't care content type.

line-bot-sdk-ruby/lib/line/bot/v2/messaging_api/api/messaging_api_blob_client.rb

Lines 238 to 276 in aebb1d3

	def set_rich_menu_image_with_http_info( # steep:ignore MethodBodyTypeMismatch
	rich_menu_id:,
	body:
	)
	path = "/v2/bot/richmenu/{richMenuId}/content"
	.gsub(/{richMenuId}/, rich_menu_id.to_s)
	response = @http_client.post(
	path: path,
	body_params: body,
	)
	case response.code.to_i
	when 200
	[response.body, 200, response.each_header.to_h]
	else
	[response.body, response.code.to_i, response.each_header.to_h]
	end
	end
	# Upload rich menu image
	# This requests to <code>POST https://api-data.line.me/v2/bot/richmenu/{richMenuId}/content</code>
	# When you want to get HTTP status code or response headers, use {#set_rich_menu_image_with_http_info} instead of this.
	#
	# @param rich_menu_id [String] The ID of the rich menu to attach the image to
	# @param body [File]
	# @see https://developers.line.biz/en/reference/messaging-api/#upload-rich-menu-image
	# @return [String, nil] when HTTP status code is 200
	# @return [String, nil] when other HTTP status code is returned. This String is HTTP response body itself.
	def set_rich_menu_image(
	rich_menu_id:,
	body:
	)
	response_body, _status_code, _headers = set_rich_menu_image_with_http_info(
	rich_menu_id: rich_menu_id,
	body: body
	)
	response_body

User can follow #238 (comment), or don't have to do anything.