aboutsummaryrefslogtreecommitdiffstats
path: root/lib/mail_handler/mail_handler.rb
Commit message (Collapse)AuthorAgeLines
* Add a default timeout of 20 mins to external commands.Louise Crow2015-03-23-1/+3
|
* Rename XXX comments with TODO:Gareth Rees2014-06-10-2/+2
| | | | Picks these up in `rake notes` and adds semantic meaning
* Treat any document/pdf attachment as if it were application/pdfMark Longair2013-12-03-1/+1
| | | | | | | | | | | | | | | | | This is a fix for issue #1232. Richard Taylor pointed out that some PDF attachments had the non-standard content-type document/pdf, and that these weren't being treated as PDFs. (Ganesh Sittampalam discovered that all of these PDFs were generated by a Lexmark X945e, according to the PDF metadata.) This commit adds an extra case to normalise_content_type to map document/pdf to application/pdf. In fact, since the upgrade of the Mail gem in ccebe3c3d6d4dc5f81 the behaviour when handling the non-standard content-type document/pdf was much better, but this commit also means that you get the right icon for the attachment, and can be cherry-picked onto older versions to fix #1232.
* Add a timeout to document conversion with wvtext - it can hang.0.13.0.2hotfix/0.13.0.2Louise Crow2013-09-04-1/+1
|
* Merge branch 'tmail-to-mail-tests' into rails-3-developMark Longair2013-05-28-3/+6
|\ | | | | | | | | | | | | | | This merge brings in a number of tests and fixes for the handling of mail under Mail / Rails 3 instead of TMail / Rails 2. Conflicts: config/initializers/alaveteli.rb
| * Suppress STDERR noise from the tnef binaryMark Longair2013-05-22-1/+1
| |
| * Ignore common TNEF attachment parsing errorsMark Longair2013-05-16-2/+5
| | | | | | | | | | | | | | This also introduces a custom error class so that we don't accidentally catch other problems. Fixes #920
| * Remove an unnecessary use of 'which' in backticksMark Longair2013-05-16-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The use of backticks in `which tnef` means that which(1) is invoked under /bin/sh, and returns the first executable matching 'tnef' in the default PATH for /bin/sh. In this usage, however, Popen also uses /bin/sh to execute the command, which would run that same executable if just a bare 'tnef' were used. In summary, I can't see any reason for the convolution of: IO.popen("#{`which tnef`.chomp} -K -C #{dir}", "wb") do |f| ... over just: IO.popen("tnef -K -C #{dir}", "wb") do |f| ... so switch to the latter.
* | Merge remote-tracking branch 'origin/hotfix/0.9.0.3' into rails-3-developLouise Crow2013-05-16-1/+2
|\ \ | |/ |/| | | | | | | | | Conflicts: Rakefile app/models/contact_mailer.rb spec/models/contact_mailer_spec.rb
| * Limit memory available to wvText, which calls elinksIan Chard2013-05-15-1/+2
| |
* | Remove Tmail and use the Mail gem under Ruby 1.8.7 as wellHenare Degan2013-02-27-11/+4
|/
* Explicitly require the mail gem when using it as a backend.Louise Crow2012-12-06-0/+1
|
* Rename _get_attachment_text_internal_one_file to ↵Louise Crow2012-12-06-2/+2
| | | | get_attachment_text_one_file as it is now an externally-accessed method of the mail handler module.
* No real need for this to be an internal function.Louise Crow2012-12-06-2/+2
|
* Move methods for getting the text out of attachments to the mail handler module.Louise Crow2012-12-06-0/+101
|
* Read and write to the file in binary mode during a TNEF conversion operation.Louise Crow2012-12-05-2/+2
|
* Move content type normalization to mail handler.Louise Crow2012-12-05-0/+24
|
* Move the action mailer dependency to the mail handler so that it's only ↵Louise Crow2012-12-04-0/+1
| | | | active in Ruby 1.8. Use mail handler methods in handle-mail-replies.
* Add some extra accessors to Mail::Message for nowLouise Crow2012-11-15-0/+1
|
* Move handling of TNEF mail attachments to mail handlerLouise Crow2012-11-15-0/+34
|
* Move TMail extensions to mail handler.Louise Crow2012-11-15-0/+1
|
* First skeletal version of separate mail handling library.Louise Crow2012-11-15-0/+16