Commit message (Collapse) | Author | Age | Lines | |
---|---|---|---|---|
* | Prevent erroneous internal error messages with multibyte characters under ↵ | Louise Crow | 2013-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.14 | Louise Crow | 2013-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 Crow | 2013-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 fallback | Mark Longair | 2013-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_number | Mark Longair | 2013-05-24 | -6/+1 |
| | ||||
* | Move the mapi requires to where they're really needed | Mark Longair | 2013-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 3 | Mark Longair | 2013-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 initializer | Mark Longair | 2013-05-16 | -1/+0 |
| | ||||
* | Change email address in header of source code to hello@mysociety.org | Matthew Landauer | 2013-03-26 | -1/+1 |
| | ||||
* | Replace use of Iconv in Ruby 1.9 to get rid of deprecation warnings | Henare Degan | 2013-03-16 | -7/+23 |
| | ||||
* | We need to reload here in Rails 3.1 (caching?) | Henare Degan | 2013-03-03 | -0/+3 |
| | ||||
* | Rename Configuration class to avoid conflict with ActiveSupport::Configurable | Henare Degan | 2013-03-03 | -4/+4 |
| | ||||
* | In Ruby 1.9.3 we need to explicitly require iconv because it's not ↵ | Henare Degan | 2013-02-28 | -1/+1 |
| | | | | implicitly loaded via Mail | |||
* | Fix calculation of attachment sizes in Ruby 1.9.3 | Henare Degan | 2013-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-spike | Henare Degan | 2013-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 Crow | 2013-02-06 | -1/+1 |
| | | | | | | | | encoded data. | |||
* | | Merge branch 'rails_xss' into rails-3-spike | Matthew Landauer | 2013-01-25 | -0/+1 |
|\| | | | | | | | | | | | | | | | Conflicts: Gemfile Gemfile.lock config/environment.rb lib/i18n_fixes.rb | |||
| * | Merge remote-tracking branch 'mysociety/develop' into rails_xss | Matthew Landauer | 2013-01-15 | -386/+90 |
| |\ | ||||
| | * | Handle case where info request doesn't have a user_name | Louise Crow | 2013-01-07 | -0/+1 |
| | | | ||||
* | | | Activerecord 3 small api change for find_or_create_by | Matthew Landauer | 2013-01-04 | -1/+1 |
| | | | ||||
* | | | Merge branch 'rails_xss' into rails-3-spike | Matthew Landauer | 2013-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 safe | Matthew Landauer | 2013-01-03 | -1/+1 |
| | | | ||||
* | | | We don't need to specify the attribute again | Henare Degan | 2012-12-12 | -1/+1 |
| |/ |/| | | | | | Makes FoiAttachment model specs pass | |||
* | | Add alternative sanitization for Ruby 1.9 | Louise Crow | 2012-12-06 | -23/+57 |
| | | ||||
* | | Convert url in comment to spec. | Louise Crow | 2012-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 Crow | 2012-12-06 | -3/+4 |
| | | | | | | | | attachment model, and hasn't necessarily been handled by TMail. | |||
* | | Reformat for line length. | Louise Crow | 2012-12-06 | -1/+3 |
| | | ||||
* | | Rename _get_attachment_text_internal_one_file to ↵ | Louise Crow | 2012-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 Crow | 2012-12-06 | -100/+2 |
| | | ||||
* | | Standard indent. | Louise Crow | 2012-12-06 | -4/+4 |
| | | ||||
* | | Write to the temp file in binary mode - we may be writing binary content, ↵ | Louise Crow | 2012-12-05 | -0/+1 |
| | | | | | | | | such as a PDF file. | |||
* | | Use the character set of the attachment to encode the string that we're ↵ | Louise Crow | 2012-12-05 | -0/+3 |
| | | | | | | | | pulling out of the file before converting it to our default encoding. | |||
* | | Move mail-specific stuff to mail handler. | Louise Crow | 2012-12-05 | -46/+1 |
| | | ||||
* | | Separate mail-specific stuff from creation/update of model instances. | Louise Crow | 2012-12-05 | -8/+12 |
| | | ||||
* | | Add a wrapper method for get_attachment_leaves. | Louise Crow | 2012-12-05 | -4/+1 |
| | | ||||
* | | Move the getting of attachment leaves to the mail handler. | Louise Crow | 2012-12-05 | -90/+1 |
| | | ||||
* | | Pass around mail object explicitly when getting attachment leaves. | Louise Crow | 2012-12-05 | -8/+8 |
| | | ||||
* | | Move part counting to the mail handler. | Louise Crow | 2012-12-05 | -53/+7 |
| | | ||||
* | | Pass explicit references to the mail when counting parts. | Louise Crow | 2012-12-05 | -13/+13 |
| | | ||||
* | | Move counters to mail object. | Louise Crow | 2012-12-05 | -5/+5 |
| | | ||||
* | | Move content type normalization to mail handler. | Louise Crow | 2012-12-05 | -27/+2 |
| | | ||||
* | | Now that the underlying methods handle the case where there is no from field ↵ | Louise Crow | 2012-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 Crow | 2012-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 Crow | 2012-12-05 | -1/+1 |
| | | ||||
* | | Rewrite spec to reflect delegation of low-level mail methods to mail ↵ | Louise Crow | 2012-12-04 | -6/+3 |
| | | | | | | | | handler, use mail handler methods. | |||
* | | Remove duplicate method that was masking what seems to be intended to be the ↵ | Louise Crow | 2012-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 Crow | 2012-12-04 | -5/+0 |
| | | ||||
* | | Remove redundant method. | Louise Crow | 2012-12-04 | -4/+0 |
| | | ||||
* | | Move method for getting the to, cc and envelope-to addresses of a mail to ↵ | Louise Crow | 2012-12-04 | -3/+1 |
| | | | | | | | | the mail handler. | |||
* | | Move the specifics of getting a From: name from a mail to the mail handler. | Louise Crow | 2012-12-03 | -4/+1 |
| | |