aboutsummaryrefslogtreecommitdiffstats
path: root/app/models/incoming_message.rb
Commit message (Collapse)AuthorAgeLines
* Prevent erroneous internal error messages with multibyte characters under ↵Louise Crow2013-07-25-2/+2
| | | | ruby 1.8 by using mb_chars for length comparison - it counts multibyte characters.
* Allow the HTML display of the body of an email to handle the case where the ↵0.11.0.14hotfix/0.11.0.14Louise Crow2013-06-19-1/+1
| | | | body is only quoted text and there is no subject.
* When extracting attachments for an incoming message and getting the body of ↵Louise Crow2013-05-29-7/+16
| | | | the main part in order to look for uuencoded text, make sure that we're getting that main part from the reparsed attachments, and not getting an obsolete attachment. Fixes #958.
* Check that display_filename matches URL part number or fallbackMark Longair2013-05-24-3/+22
| | | | | | | | | | | If the display_filename of the attachment found from the URL part number doesn't match the passed in display_filename then the email may have been reparsed, causing a reordering. In that case, look to see if there is another attachment that uniquely matches that filename, and, if so, return that other attachment. If no matching uniquely matching filename is found, redirect to the incoming message, rather than returning a 404.
* Refactor IncomingMessage.get_attachment_by_url_part_numberMark Longair2013-05-24-6/+1
|
* Move the mapi requires to where they're really neededMark Longair2013-05-22-2/+0
| | | | | | | Handling of outlook-packed attachments would fail from rake tasks or in the console without requiring 'mapi/msg' and 'mapi/convert' beforehand. Instead, require them in the source file where they're actually used.
* Avoid an FoiAttachment validation failure under Rails 3Mark Longair2013-05-16-0/+1
| | | | | | | | | | | Under Rails 3, the uudecoded FoiAttachment in this test fails validation at the self.save! in IncomingMessage.parse_raw_email, although the FoiAttachment has been correctly created and saved to the database in _uudecode_and_save_attachments. Forcing a reload=true on self.foi_attachments fixes this. Thanks to Louise Crow for finding the fix for this problem.
* Move "require 'alaveteli_file_types'" into the initializerMark Longair2013-05-16-1/+0
|
* Change email address in header of source code to hello@mysociety.orgMatthew Landauer2013-03-26-1/+1
|
* Replace use of Iconv in Ruby 1.9 to get rid of deprecation warningsHenare Degan2013-03-16-7/+23
|
* We need to reload here in Rails 3.1 (caching?)Henare Degan2013-03-03-0/+3
|
* Rename Configuration class to avoid conflict with ActiveSupport::ConfigurableHenare Degan2013-03-03-4/+4
|
* In Ruby 1.9.3 we need to explicitly require iconv because it's not ↵Henare Degan2013-02-28-1/+1
| | | | implicitly loaded via Mail
* Fix calculation of attachment sizes in Ruby 1.9.3Henare Degan2013-02-27-3/+0
| | | | | | | | force_encoding is generally not a good idea and I can't see what it's trying to achieve here so... Revert "Use the character set of the attachment to encode the string that we're pulling out of the file before converting it to our default encoding." This reverts commit 3729f2053d4c04396d440a9c368bed174e9c9605.
* Merge branch 'develop' into rails-3-spikeHenare Degan2013-02-15-1/+1
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: Gemfile Gemfile.lock app/controllers/general_controller.rb app/controllers/track_controller.rb app/models/outgoing_message.rb app/models/public_body.rb app/models/user.rb app/views/general/frontpage.rhtml config/environment.rb config/initializers/inflections.rb config/initializers/mime_types.rb db/migrate/094_remove_old_tags_foreign_key.rb lib/timezone_fixes.rb spec/models/request_mailer_spec.rb spec/views/request/list.rhtml_spec.rb
| * Rescue an invalid character exception when handling what appears to be badly ↵Louise Crow2013-02-06-1/+1
| | | | | | | | encoded data.
* | Merge branch 'rails_xss' into rails-3-spikeMatthew Landauer2013-01-25-0/+1
|\| | | | | | | | | | | | | | | Conflicts: Gemfile Gemfile.lock config/environment.rb lib/i18n_fixes.rb
| * Merge remote-tracking branch 'mysociety/develop' into rails_xssMatthew Landauer2013-01-15-386/+90
| |\
| | * Handle case where info request doesn't have a user_nameLouise Crow2013-01-07-0/+1
| | |
* | | Activerecord 3 small api change for find_or_create_byMatthew Landauer2013-01-04-1/+1
| | |
* | | Merge branch 'rails_xss' into rails-3-spikeMatthew Landauer2013-01-04-1/+1
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: Gemfile Gemfile.lock app/views/request/_hidden_correspondence.rhtml app/views/request/hidden.rhtml app/views/request/new_please_describe.rhtml app/views/user/show.rhtml lib/i18n_fixes.rb
| * | html content of messages and comments are html safeMatthew Landauer2013-01-03-1/+1
| | |
* | | We don't need to specify the attribute againHenare Degan2012-12-12-1/+1
| |/ |/| | | | | Makes FoiAttachment model specs pass
* | Add alternative sanitization for Ruby 1.9Louise Crow2012-12-06-23/+57
| |
* | Convert url in comment to spec.Louise Crow2012-12-06-1/+0
| | | | | | | | | | | | Conflicts: lib/mail_handler/backends/tmail_backend.rb
* | Update comments to reflect the fact that text is coming from an foi ↵Louise Crow2012-12-06-3/+4
| | | | | | | | attachment model, and hasn't necessarily been handled by TMail.
* | Reformat for line length.Louise Crow2012-12-06-1/+3
| |
* | Rename _get_attachment_text_internal_one_file to ↵Louise Crow2012-12-06-2/+3
| | | | | | | | get_attachment_text_one_file as it is now an externally-accessed method of the mail handler module.
* | Move methods for getting the text out of attachments to the mail handler module.Louise Crow2012-12-06-100/+2
| |
* | Standard indent.Louise Crow2012-12-06-4/+4
| |
* | Write to the temp file in binary mode - we may be writing binary content, ↵Louise Crow2012-12-05-0/+1
| | | | | | | | such as a PDF file.
* | Use the character set of the attachment to encode the string that we're ↵Louise Crow2012-12-05-0/+3
| | | | | | | | pulling out of the file before converting it to our default encoding.
* | Move mail-specific stuff to mail handler.Louise Crow2012-12-05-46/+1
| |
* | Separate mail-specific stuff from creation/update of model instances.Louise Crow2012-12-05-8/+12
| |
* | Add a wrapper method for get_attachment_leaves.Louise Crow2012-12-05-4/+1
| |
* | Move the getting of attachment leaves to the mail handler.Louise Crow2012-12-05-90/+1
| |
* | Pass around mail object explicitly when getting attachment leaves.Louise Crow2012-12-05-8/+8
| |
* | Move part counting to the mail handler.Louise Crow2012-12-05-53/+7
| |
* | Pass explicit references to the mail when counting parts.Louise Crow2012-12-05-13/+13
| |
* | Move counters to mail object.Louise Crow2012-12-05-5/+5
| |
* | Move content type normalization to mail handler.Louise Crow2012-12-05-27/+2
| |
* | Now that the underlying methods handle the case where there is no from field ↵Louise Crow2012-12-05-2/+2
| | | | | | | | content by returning nil, I don't think we need to rescue a NoMethodError - this would have originally been returned by calling spec on from_addrs[0] when it was nil.
* | Use the binary_output option to have the strings returned by ↵Louise Crow2012-12-05-8/+17
| | | | | | | | AlaveteliExternalCommand be transcoded using the default internal and external encodings in Ruby 1.9.
* | Use mail handler to get the content type of each email part.Louise Crow2012-12-05-1/+1
| |
* | Rewrite spec to reflect delegation of low-level mail methods to mail ↵Louise Crow2012-12-04-6/+3
| | | | | | | | handler, use mail handler methods.
* | Remove duplicate method that was masking what seems to be intended to be the ↵Louise Crow2012-12-04-30/+1
| | | | | | | | real valid_to_reply_to? method - which accesses cached values. Replacing the _calculate_valid_to_reply_to logic with the code that will have actually been running.
* | Remove obsolete comment.Louise Crow2012-12-04-5/+0
| |
* | Remove redundant method.Louise Crow2012-12-04-4/+0
| |
* | Move method for getting the to, cc and envelope-to addresses of a mail to ↵Louise Crow2012-12-04-3/+1
| | | | | | | | the mail handler.
* | Move the specifics of getting a From: name from a mail to the mail handler.Louise Crow2012-12-03-4/+1
| |