Attachment Follow Up does not work with pdf files

wucherpfennig · August 26, 2024, 2:05pm

Infos:

Used Zammad version: latest
Used Zammad installation type: package
Operating system: debian
Browser + version: firefox

Expected behavior:

search for follow ups in pdf attachments

Actual behavior:

does not search in pdf attachments
works fine with *.txt files

controll1:

manually searching for the ticket number (from the attachment) results in two results: 1 x ticket and 1 x NEW ticket with attachment

controll2:

doing the test again with a dotx file: same behavoir. follow up does not work but search resulst in 2 tickets: 1 x old ticket and 1 NEW ticket with attachment

therefore the I assume that indexing is working but no used to do the follow up look up?

Steps to reproduce the behavior:

create a ticket
send an email with pdf attachment with text [T#123123]
system creates a new ticket instead of follow up

wucherpfennig · August 26, 2024, 2:23pm

Well I think this is how it goes… if I am not mistaken this feature works only with text/plain and text/html which renders the feature somehow useless while working with reports etc.

could anybody confirm?

github.com

zammad/zammad/blob/03b48dfeeb9e61fd9dafca5fd9d2f2c85eeba9c8/app/models/channel/filter/follow_up_check.rb#L32C1-L58C8


      
          # get ticket# from attachment
          if setting.include?('attachment') && mail[:attachments]
            mail[:attachments].each do |attachment|
              next if attachment[:data].blank?
              next if attachment[:preferences].blank?
              next if attachment[:preferences]['Mime-Type'].blank?
          
              if %r{text/html}i.match?(attachment[:preferences]['Mime-Type'])
                begin
                  text = attachment[:data].html2text
                  ticket = Ticket::Number.check(text)
                rescue => e
                  Rails.logger.error e
                end
              end
          
              if %r{text/plain}i.match?(attachment[:preferences]['Mime-Type'])
                ticket = Ticket::Number.check(attachment[:data])
              end

This file has been truncated. show original

wucherpfennig · August 26, 2024, 2:45pm

well, maybe there is a solution: GitHub - yob/pdf-reader: The PDF::Reader library implements a PDF parser conforming as much as possible to the PDF specification from Adobe.