Opened 8 years ago
Closed 7 years ago
#12683 closed defect (fixed)
RSS feeds truncated or broken
Reported by: | Owned by: | Christian Boos | |
---|---|---|---|
Priority: | normal | Milestone: | 1.3.2 |
Component: | query system | Version: | 1.3dev |
Severity: | normal | Keywords: | rss |
Cc: | Branch: | ||
Release Notes: | |||
API Changes: | |||
Internal Changes: |
Description
I note the RSS feeds on your live trac.edgewall.org site currently appear truncated (sometimes empty).
For example:
- Perform a ticket query, e.g. Summary contains RSS. (about 145 results)
- Click the "RSS Feed" link at the bottom.
Only 5 results are shown.
Attachments (0)
Change History (9)
comment:1 by , 8 years ago
comment:2 by , 8 years ago
Component: | notification → query system |
---|---|
Milestone: | → 1.3.2 |
follow-up: 4 comment:3 by , 8 years ago
The same result even if private browsing is used. It doesn't depend on the session. Also, /query?summary=~RSS&format=rss&order=priority&max=9999999
returns 5 entries.
comment:4 by , 8 years ago
Replying to Jun Omae:
The same result even if private browsing is used. It doesn't depend on the session. Also,
/query?summary=~RSS&format=rss&order=priority&max=9999999
returns 5 entries.
Well, I have all the 145 entries with that…
comment:5 by , 8 years ago
The description element in RSS cannot contain html tags. We should escape it.
- 1.3.2dev
- https://www.feedvalidator.org/check.cgi?url=https%3A%2F%2Ftrac.edgewall.org%2Fquery%3Fsummary%3D%7ERSS%26format%3Drss%26order%3Dpriority%26max%3D10
- 1.0 (demo)
- https://www.feedvalidator.org/check.cgi?url=https%3A%2F%2Ftrac.edgewall.org%2Fdemo-1.0%2Fquery%3Fstatus%3D%21closed%26max%3D10%26format%3Drss
comment:7 by , 8 years ago
Version: | → 1.3dev |
---|
Should be fixed with r15521.
Ideally there should be a few functional tests for this, as the RSS functionality remains largely uncovered. Leaving the ticket open for that.
comment:8 by , 8 years ago
Milestone: | 1.3.2 → 1.3.3 |
---|
comment:9 by , 7 years ago
Milestone: | 1.3.3 → 1.3.2 |
---|---|
Resolution: | → fixed |
Status: | assigned → closed |
Moving this to milestone:1.3.2 since it was fixed there. Would be great to have more test coverage.
I think it takes the max rows info from the session. Try changing that or setting it explicitly in the URL (append
&max=100
).