プロジェクト

全般

プロフィール

Vote #67058

完了

projects.atom with required authentication

Admin Redmine さんがほぼ2年前に追加. ほぼ2年前に更新.

ステータス:
Closed
優先度:
通常
担当者:
-
カテゴリ:
Feeds_18
対象バージョン:
開始日:
2010/04/14
期日:
進捗率:

100%

予定工数:
category_id:
18
version_id:
19
issue_org_id:
5317
author_id:
11501
assigned_to_id:
5
comments:
14
status_id:
5
tracker_id:
1
plus1:
0
affected_version:
closed_on:
affected_version_id:
17
ステータス-->[Closed]

説明

We are running a redmine deployment with required authentication.
Most RSS feeds seem to work in this setup, but the link to the feed from
the projects page ends up at the login page, when not requested from
the browser with a valid session.

Used software:
mysql Ver 14.14 Distrib 5.1.37, for debian-linux-gnu (x86_64)
ruby 1.8.7
rails 2.3.5

The message in the logs boils down to:

Filter chain halted as [:check_if_login_required] rendered_or_redirected.

I searched for this issue, but did only find #2078, which in fact would be also nice to have.


journals

Hi there,
just wanted to report, that this issue still persists after the upgrade to 0.9.4.
--------------------------------------------------------------------------------
Yep,
I confirm that it is still present in 0.9.4.
--------------------------------------------------------------------------------
Could you tell us more about your setups, especially the source of your redmine (i.e. downloaded from redmine.org or installed from a debian package)?
--------------------------------------------------------------------------------
I´m sorry but I cannot give you this information because my hoster installed it.

I see what I can do to get it ;)
--------------------------------------------------------------------------------
Moritz: I have a somewhat sure way to find it out if you have access to the redmine files on your server. Have a look at @redmine_dir/config/initializers/40_email.rb@, if it looks for the file email.yml in @redmine_dir/config/email.yml@ it's stock, if it looks in @/etc/redmine@ it's the deb.
--------------------------------------------------------------------------------
Ok, thanks.
It looks for config/email.yml, so it isnt the db package.
--------------------------------------------------------------------------------
Just confirmed that on r3764.
--------------------------------------------------------------------------------
Add @:index@ to source:trunk/app/controllers/projects_controller.rb#L30, i.e. change:

<pre> accept_key_auth :activity</pre>

to:

<pre> accept_key_auth :activity, :index</pre>

@Eric: Could you push that to trunk?
--------------------------------------------------------------------------------
Felix Schäfer wrote:
> @Eric: Could you push that to trunk?

And while we're at it, I think it wouldn't hurt having the ATOM link for the project index in the header, i.e. add the following to source:trunk/app/views/projects/index.rhtml :

<pre><% content_for :header_tags do %>
<%= auto_discovery_link_tag(:atom, {:action => 'index', :format => 'atom', :key => User.current.rss_key}) %>
<% end %></pre>

Thanks :-)
--------------------------------------------------------------------------------
Hi,
sorry for the somewhat late reply, and thanks for looking at this issue.
I'm following the SVN repository, but only update to tagged releases, so
no debian specific stuff is involved.
I applied the both modifications (to 0.9.4), and the problem is gone.

Thanks a lot!
--------------------------------------------------------------------------------
h3. To reproduce from a clean install of trunk:

# Administration -> Settings -> Authentication -> Check "Authentication Required." -> Click "Save"
# Create a test project
# Go to list of projects
** There should be no "Atom" icon in the browser address bar
# Click the "Atom" link at the bottom of the projects page
** Should see the Atom feed
# Copy the URL from the address bar
# Click "Log Out"
# Paste the URL back in the address bar
** Should see the login page

h3. After applying the patch:

# Log in and get the link to the Atom feed from the projects list
** Should see the RSS/Atom auto-discovery icon in the address bar (although it looks like Chrome doesn't work)
# Log back out
** Should see the Atom feed now

--------------------------------------------------------------------------------
Split up the authentication fix from the auto_discovery fix.
--------------------------------------------------------------------------------
Committed both the auto discovery link and key authentication in r3776 and r3777. Thank you.
--------------------------------------------------------------------------------
Merged into 0.9-stable.
--------------------------------------------------------------------------------


related_issues

relates,Closed,6132,Allow Key based authentication in the Boards atom feed

Admin Redmine さんがほぼ2年前に更新

  • カテゴリFeeds_18 にセット
  • 対象バージョン0.9.5_19 にセット

他の形式にエクスポート: Atom PDF

いいね!0
いいね!0