diff --git a/app/models/mailer.rb b/app/models/mailer.rb
index 070f7e5..397807d 100644
--- a/app/models/mailer.rb
+++ b/app/models/mailer.rb
@@ -183,9 +183,13 @@ class Mailer < ActionMailer::Base
super
set_language_if_valid Setting.default_language
from Setting.mail_from
- default_url_options[:host] = Setting.host_name
+
+ # URL options
+ h = Setting.host_name
+ h = h.to_s.gsub(%r{\/.*$}, '') unless ActionController::AbstractRequest.relative_url_root.blank?
+ default_url_options[:host] = h
default_url_options[:protocol] = Setting.protocol
- default_url_options[:skip_relative_url_root] = true
+
# Common headers
headers 'X-Mailer' => 'Redmine',
'X-Redmine-Host' => Setting.host_name,
diff --git a/test/unit/mailer_test.rb b/test/unit/mailer_test.rb
index e3a21a5..1fd43af 100644
--- a/test/unit/mailer_test.rb
+++ b/test/unit/mailer_test.rb
@@ -38,6 +38,54 @@ class MailerTest < Test::Unit::TestCase
# link to a changeset
assert mail.body.include?('r2')
end
+
+ def test_generated_links_with_prefix
+ relative_url_root = ActionController::AbstractRequest.relative_url_root
+ ActionMailer::Base.deliveries.clear
+ Setting.host_name = 'mydomain.foo/rdm'
+ Setting.protocol = 'http'
+ ActionController::AbstractRequest.relative_url_root = '/rdm'
+
+ journal = Journal.find(2)
+ assert Mailer.deliver_issue_edit(journal)
+
+ mail = ActionMailer::Base.deliveries.last
+ assert_kind_of TMail::Mail, mail
+ # link to the main ticket
+ assert mail.body.include?('Bug #1: Can\'t print recipes')
+
+ # link to a referenced ticket
+ assert mail.body.include?('#2')
+ # link to a changeset
+ assert mail.body.include?('r2')
+ ensure
+ # restore it
+ ActionController::AbstractRequest.relative_url_root = relative_url_root
+ end
+
+ def test_generated_links_with_prefix_and_no_relative_url_root
+ relative_url_root = ActionController::AbstractRequest.relative_url_root
+ ActionMailer::Base.deliveries.clear
+ Setting.host_name = 'mydomain.foo/rdm'
+ Setting.protocol = 'http'
+ ActionController::AbstractRequest.relative_url_root = nil
+
+ journal = Journal.find(2)
+ assert Mailer.deliver_issue_edit(journal)
+
+ mail = ActionMailer::Base.deliveries.last
+ assert_kind_of TMail::Mail, mail
+ # link to the main ticket
+ assert mail.body.include?('Bug #1: Can\'t print recipes')
+
+ # link to a referenced ticket
+ assert mail.body.include?('#2')
+ # link to a changeset
+ assert mail.body.include?('r2')
+ ensure
+ # restore it
+ ActionController::AbstractRequest.relative_url_root = relative_url_root
+ end
def test_plain_text_mail
Setting.plain_text_mail = 1