Defect #19537
closed
Broken HTML sanitizer refence breaks email receiving
Added by Phil Stewart over 9 years ago.
Updated over 9 years ago.
Description
HTML emails fail silently when receiving emails via IMAP due to broken HTML sanitizer reference in MailHandler.full_sanitizer on line 449 of app/models/mail_handler.rb:
def self.full_sanitizer
@full_sanitizer ||= HTML::FullSanitizer.new
end
Changing this line to @full_sanitizer ||= Rails::Html::FullSanitizer.new works, although it didn't do a particularly good job on the stuck email I'd sent from Outlook.
Redmine: 3.0.1
Rails: 4.2.0
Ruby: 2.2.0
Files
This line @full_sanitizer ||= Rails::Html::FullSanitizer.new works:
Ruby 2.0.0
Rails 4.2.0
Redmine 3.0.1
And normally works on import mail sent by MS Outlook 2013.
This is a new implementation of rails sanitizer, This did not work, So using the old deprecated one, through a GEM.
gem 'rails-deprecated_sanitizer'
Until this is corrected with the rails sanitizer...! :)
I've created a Gemfile.local file in my install and added gem 'rails-deprecated_sanitizer', which works nicely without having to change MailHandler over to the new sanitizer.
- Target version set to 3.0.2
- Subject changed from Broken HTML sanitizer refence breaks redmine:email:receive_imap to Broken HTML sanitizer refence breaks email receiving
- Status changed from New to Closed
- Assignee set to Jean-Philippe Lang
- Resolution set to Fixed
Fixed in r14219, the default Rails sanitizer is now used.
The text striping is not handled.
I am getting some unwanted Outlook mail tags.
Please let me know if this needs to be reported as another bug ?
Rupesh J wrote:
The text striping is not handled.
I am getting some unwanted Outlook mail tags.
Did it work on Redmine 2.x?
Please let me know if this needs to be reported as another bug ?
If it is regression of Redmine 3.0,
please create new issue because 3.0.2 was released.
If it is not regression of Redmine 3.0,
I think it is duplicate of #13209.
Ok, I will create a regression bug.
It worked in
- 2.6.x
- 3.0.1 using the gem 'rails-deprecated_sanitizer'
Thank you for creating #19737.
Also available in: Atom
PDF
Updated by 
Updated by 
Updated by 
Updated by Toshi MARUYAMA 
Updated by 