Defect #10535

Atom feeds for the Project Activity contain incorrect hostname

Added by Artem Sheremet over 8 years ago. Updated 2 months ago.

Status:ClosedStart date:
Priority:NormalDue date:
Assignee:Go MAEDA% Done:

0%

Category:Feeds
Target version:4.2.0
Resolution:Fixed Affected version:1.3.0

Description

The link in the Atom feed (at least for Project Activity) contains a hostname detected by Rails, but not the one specified in settings.
This makes link unusable if the requests to Rails are proxified.

Versions:

Redmine:

SVN: http://redmine.rubyforge.org/svn/branches/1.3-stable revision 9240

Env:

Ruby version              1.8.7 (x86_64-linux)
RubyGems version          1.3.7
Rack version              1.1.2
Rails version             2.3.14
Active Record version     2.3.14
Active Resource version   2.3.14
Action Mailer version     2.3.14
Active Support version    2.3.14
Application root          /opt/redmine
Environment               production
Database adapter          mysql
Database schema version   20110902000000

fix-10535-without-test.patch Magnifier (1.6 KB) Go MAEDA, 2020-05-19 10:11

test-10535.patch Magnifier (6.28 KB) Go MAEDA, 2020-05-19 14:11

Associated revisions

Revision 19810
Added by Go MAEDA 2 months ago

URLs in atom feeds don't take into account Setting.protocol and Setting.hostname (#10535).

Patch by Go MAEDA.

History

#1 Updated by Go MAEDA 3 months ago

Confirmed the issue. I think the links in the Atom feed should respect Setting.protocol and Setting.host_name.

The attached is a workaround for the issue.

#2 Updated by Go MAEDA 3 months ago

  • File test-10535.patchMagnifier added
  • Target version set to Candidate for next major release

Updated tests.

#3 Updated by Go MAEDA 2 months ago

  • Target version changed from Candidate for next major release to 4.2.0

Setting the target version to 4.2.0.

#4 Updated by Go MAEDA 2 months ago

  • Status changed from Confirmed to Closed
  • Assignee set to Go MAEDA
  • Resolution set to Fixed

Committed the patch.

Also available in: Atom PDF