WordPress.org

Making WordPress.org

Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#21 closed defect (fixed)

RSS feed links asking user to log in

Reported by: johnbillion Owned by: nacin
Milestone: Priority: low
Component: Trac Keywords:
Cc:

Description

The RSS Feed link at the bottom of ticket listing screens on Trac is pointing to /login. After logging in (why?) the resulting redirect 404s.

At a quick glance it looks like it's affecting all of our Tracs.

Change History (9)

#1 @Otto42
6 years ago

  • Cc nacin added

cc'ing nacin on this, as the trac theme master. :)

#2 @nacin
6 years ago

Yeah but I have no idea what is going on here. :-(

#3 @nacin
6 years ago

Dug into this. If you are logged out, you see a standard RSS link. If you are logged in, you see a /login redirect. This is to ensure you can access an RSS feed of a private resource. In the case of our Tracs on WordPress.org, there is no private information — it would be totally fine to just not send any of these through /login. Unfortunately there is no way to turn this off in Trac.

The redirection code in Trac then declines to decode the URL-encoded referer parameter. I am not sure why that occurs, or why it might work on other Tracs. I can't find any issues upstream.

#4 @netweb
6 years ago

Just came across this myself looking to subscribe via RSS to a particular #WP ticket.

Leaving aside to logged in/logged out issue, looking at the link structure itself for this ticket #21

The RSS link at the bottom of this page:
<a rel="nofollow" href="/login?referer=%2Fticket%2F21%3Fformat%3Drss" class="rss">RSS Feed</a>

As opened in Google Chrome (after accepting login credentials)
https://meta.trac.wordpress.org/%2Fticket%2F21%3Fformat%3Drss

Results in:

Not Found

The requested URL //ticket/21?format=rss was not found on this server.

Apache Server at meta.trac.wordpress.org Port 443

The only way I can actually get the RSS feed is to manually edit the URL to:
https://meta.trac.wordpress.org/ticket/21?format=rss
or
http://meta.trac.wordpress.org/ticket/21?format=rss

#5 @netweb
6 years ago

  • Cc netweb added

#6 @nacin
6 years ago

  • Owner set to nacin
  • Resolution set to fixed
  • Status changed from new to closed

In 188:

Trac: Fix RSS feed links. fixes #21.

#7 @nacin
6 years ago

The same links used in "Download in other formats:" are used to generate <link> elements in the <head>. I played with a Genshi replacement first (as seen in the rest of site.html), but [188] was much smoother.

#8 @nacin
6 years ago

In 189:

Trac: Better exception handling for [188]. see #21.

#9 @nacin
6 years ago

In 190:

Trac: Catch KeyError exception too. see [189], [188]. see #21.

Note: See TracTickets for help on using tickets.