Facebook Messenger sticker messages no longer break message ingestion. Meta recently changed its webhook payloads so a sticker now arrives as a new `sticker` attachment type (alongside the existing `image` attachment during the transition period). Chatwoot didn't recognise `sticker` as a valid attachment file type, so the webhook job crashed and the message — and any others in the same batch — failed to sync. Stickers now appear in the conversation as a single image, just like before. Fixes https://linear.app/chatwoot/issue/PLA-177 ## How to reproduce 1. Connect a Facebook Page inbox. 2. Send a sticker from Messenger to that page. 3. Before this change: the `Webhooks::FacebookEventsJob` raises `ArgumentError: 'sticker' is not a valid file_type` and the message is dropped. 4. After this change: the sticker shows up in the conversation as a single image attachment. --------- Co-authored-by: Muhsin <12408980+muhsin-k@users.noreply.github.com> Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
154 lines
5.4 KiB
Ruby
154 lines
5.4 KiB
Ruby
class Messages::Messenger::MessageBuilder
|
|
include ::FileTypeHelper
|
|
|
|
def process_attachment(attachment)
|
|
# This check handles very rare case if there are multiple files to attach with only one unsupported file
|
|
return if unsupported_file_type?(attachment['type'])
|
|
|
|
params = attachment_params(attachment)
|
|
# During Meta's sticker webhook transition, a sticker message carries both an `image`
|
|
# and a `sticker` attachment pointing to the same URL. Skip the redundant sticker so it
|
|
# isn't attached twice, while still storing legitimate duplicate attachments of other types.
|
|
return if duplicate_sticker?(attachment, params[:external_url])
|
|
|
|
attachment_obj = @message.attachments.new(params.except(:remote_file_url))
|
|
attachment_obj.save!
|
|
if facebook_reel?(attachment)
|
|
update_facebook_reel_content(attachment)
|
|
elsif params[:remote_file_url]
|
|
attach_file(attachment_obj, params[:remote_file_url])
|
|
end
|
|
fetch_attachment_links(attachment_obj)
|
|
update_attachment_file_type(attachment_obj)
|
|
end
|
|
|
|
def fetch_attachment_links(attachment_obj)
|
|
fetch_story_link(attachment_obj) if attachment_obj.file_type == 'story_mention'
|
|
fetch_ig_story_link(attachment_obj) if attachment_obj.file_type == 'ig_story'
|
|
fetch_ig_post_link(attachment_obj) if attachment_obj.file_type == 'ig_post'
|
|
end
|
|
|
|
def attach_file(attachment, file_url)
|
|
attachment_file = Down.download(
|
|
file_url
|
|
)
|
|
attachment.file.attach(
|
|
io: attachment_file,
|
|
filename: attachment_file.original_filename,
|
|
content_type: attachment_file.content_type
|
|
)
|
|
# The Attachment row is saved before the blob is attached, so the
|
|
# after_create_commit broadcast bails on `file.attached?`. Re-fire here
|
|
# for audio so the bubble updates without waiting on transcription.
|
|
attachment.message&.reload&.send_update_event if attachment.file_type.to_sym == :audio
|
|
end
|
|
|
|
def attachment_params(attachment)
|
|
file_type = normalize_file_type(attachment['type'])
|
|
params = { file_type: file_type, account_id: @message.account_id }
|
|
|
|
if [:image, :file, :audio, :video, :share, :story_mention, :ig_reel, :ig_post, :ig_story].include? file_type
|
|
params.merge!(file_type_params(attachment))
|
|
elsif file_type == :location
|
|
params.merge!(location_params(attachment))
|
|
elsif file_type == :fallback
|
|
params.merge!(fallback_params(attachment))
|
|
end
|
|
|
|
params
|
|
end
|
|
|
|
def file_type_params(attachment)
|
|
# Handle different URL field names for different attachment types
|
|
url = case attachment['type'].to_sym
|
|
when :ig_story
|
|
attachment['payload']['story_media_url']
|
|
else
|
|
attachment['payload']['url']
|
|
end
|
|
|
|
{
|
|
external_url: url,
|
|
remote_file_url: url
|
|
}
|
|
end
|
|
|
|
def update_attachment_file_type(attachment)
|
|
return if @message.reload.attachments.blank?
|
|
return unless attachment.file_type == 'share' || attachment.file_type == 'story_mention'
|
|
|
|
attachment.file_type = file_type(attachment.file&.content_type)
|
|
attachment.save!
|
|
end
|
|
|
|
def fetch_story_link(attachment)
|
|
message = attachment.message
|
|
result = get_story_object_from_source_id(message.source_id)
|
|
|
|
return if result.blank?
|
|
|
|
story_id = result['story']['mention']['id']
|
|
story_sender = result['from']['username']
|
|
message.content_attributes[:story_sender] = story_sender
|
|
message.content_attributes[:story_id] = story_id
|
|
message.content_attributes[:image_type] = 'story_mention'
|
|
message.content = I18n.t('conversations.messages.instagram_story_content', story_sender: story_sender)
|
|
message.save!
|
|
end
|
|
|
|
def fetch_ig_story_link(attachment)
|
|
message = attachment.message
|
|
# For ig_story, we don't have the same API call as story_mention, so we'll set it up similarly but with generic content
|
|
message.content_attributes[:image_type] = 'ig_story'
|
|
message.content = I18n.t('conversations.messages.instagram_shared_story_content')
|
|
message.save!
|
|
end
|
|
|
|
def fetch_ig_post_link(attachment)
|
|
message = attachment.message
|
|
message.content_attributes[:image_type] = 'ig_post'
|
|
message.content = I18n.t('conversations.messages.instagram_shared_post_content')
|
|
message.save!
|
|
end
|
|
|
|
# This is a placeholder method to be overridden by child classes
|
|
def get_story_object_from_source_id(_source_id)
|
|
{}
|
|
end
|
|
|
|
private
|
|
|
|
# Facebook may send attachment types that don't directly match our file_type enum.
|
|
# Map known aliases to their canonical enum values.
|
|
FACEBOOK_FILE_TYPE_MAP = { reel: :ig_reel, sticker: :image }.freeze
|
|
|
|
def normalize_file_type(type)
|
|
sym = type.to_sym
|
|
FACEBOOK_FILE_TYPE_MAP.fetch(sym, sym)
|
|
end
|
|
|
|
def duplicate_sticker?(attachment, url)
|
|
return false unless attachment['type'].to_sym == :sticker
|
|
return false if url.blank?
|
|
|
|
@message.attachments.any? { |existing| existing.external_url == url }
|
|
end
|
|
|
|
# Facebook sends reel URLs as webpage links (facebook.com/reel/...) rather than
|
|
# direct video URLs. Downloading these yields HTML, not video content.
|
|
def facebook_reel?(attachment)
|
|
attachment['type'].to_sym == :reel
|
|
end
|
|
|
|
def update_facebook_reel_content(attachment)
|
|
url = attachment.dig('payload', 'url')
|
|
return if url.blank?
|
|
|
|
@message.update!(content: url) if @message.content.blank?
|
|
end
|
|
|
|
def unsupported_file_type?(attachment_type)
|
|
[:template, :unsupported_type, :ephemeral].include? attachment_type.to_sym
|
|
end
|
|
end
|