Closed Bug 891372 Opened 11 years ago Closed 11 years ago

RSS feed does not get latest posts

Categories

(MailNews Core :: Feed Reader, defect)

x86_64
Linux
defect
Not set
normal

Tracking

(thunderbird24 fixed, thunderbird25 fixed)

RESOLVED FIXED
Thunderbird 26.0
Tracking Status
thunderbird24 --- fixed
thunderbird25 --- fixed

People

(Reporter: a.vankaam, Assigned: alta88)

Details

Attachments

(1 file, 1 obsolete file)

User Agent: Opera/9.80 (X11; Linux x86_64) Presto/2.12.388 Version/12.15

Steps to reproduce:

Edit, account settings, account actions, add other account, blogs & news feeds, next, next, finish.
Right click the created Blogs & News feeds and select subscribe
Enter URL: http://www.gamingonlinux.com/article_rss.php
Press ADD
close dialog



Actual results:

It will download the last 15 articles, at this moment when I write this the latest entry it shows is "Limit Theory Developer Update on Linux Port"  which was from 3 days ago.  


Expected results:

It should have gotten the 8 entries which has been posted since then but it does not.

Going with a web browser to http://www.gamingonlinux.com/article_rss.php shows them.
Using opera RSS feeds shows them.
Using Akregator RSS reader shows them.
Using the thunderbird add on bamboo shows them.

Get messages does not get anything new, restarting to make it check on restart does not get anything new

Feeds.logging.console set to debug shows:

2013-07-09 16:55:50	Feeds	DEBUG	downloadFeed: START x/# foldername:uri - 1/1 GamingOnLinux.com Latest articles:mailbox://nobody@Feeds-2/GamingOnLinux.com%20Latest%20articles
2013-07-09 16:55:50	Feeds	DEBUG	downloadFeed: CONTINUE foldername:urlArray - GamingOnLinux.com Latest articles:http://www.gamingonlinux.com/article_rss.php
2013-07-09 16:55:50	Feeds	DEBUG	downloadFeed: DOWNLOAD feed url - http://www.gamingonlinux.com/article_rss.php
2013-07-09 16:55:50	Feeds	DEBUG	downloadFeed: Finished with folder - GamingOnLinux.com Latest articles
2013-07-09 16:55:50	Feeds	DEBUG	Feed.onDownloaded: got a download - http://www.gamingonlinux.com/article_rss.php
2013-07-09 16:55:50	Feeds	DEBUG	FeedParser.parseFeed: type:url - RSS_2.0 : http://www.gamingonlinux.com/article_rss.php
2013-07-09 16:55:50	Feeds	DEBUG	FeedParser.parseAsRSS2: channel nsURI - 
2013-07-09 16:55:50	Feeds	DEBUG	Feed.invalidateItems: for url - http://www.gamingonlinux.com/article_rss.php
2013-07-09 16:55:50	Feeds	DEBUG	FeedParser.parseAsRSS2: items to parse - 15
2013-07-09 16:55:51	Feeds	DEBUG	Feed.removeInvalidItems: for url - http://www.gamingonlinux.com/article_rss.php
2013-07-09 16:55:51	Feeds	DEBUG	downloaded: Update errorCode:feedName:folder - 0 : GamingOnLinux.com Latest articles : /media/realcrypt1/Thunderbird/default/Mail/Feeds-2-1/GamingOnLinux.com Latest articles
this works for me (linux, Tb25).  as of this moment, all 15 current articles were downloaded, starting from Project 64 to Infested Planet (newest), which is what the file contains.
but, in fact, updates do not work.  the stale cached file is being used..
Status: UNCONFIRMED → NEW
Ever confirmed: true
Attached patch patch (obsolete) — Splinter Review
Assignee: nobody → alta88
Attachment #773308 - Flags: review?(mbanner)
Comment on attachment 773308 [details] [diff] [review]
patch

[Approval Request Comment]

this is borderline data lossy; risk is low.
Attachment #773308 - Flags: approval-comm-aurora?
thank you for solving this as fast as you did.  Is there anyway to incorporate this patch in TB17 ? don't see a feed.js anything so assume its compiled in somewhere
(In reply to a.vankaam from comment #5)
> thank you for solving this as fast as you did.  Is there anyway to
> incorporate this patch in TB17 ? don't see a feed.js anything so assume its
> compiled in somewhere

it's not up to me, but it could go into a .0x of 17, though there may never be one.  also, it's hard to say how widespread it is but i suspect it may have to do with how that server sets headers.  for one, they don't respond with 304 on a failed If-Modified-Since.

anyway, you could unzip omni.ja, find Feed.js, and insert the 2 lines, and rearchive (or find an in place archive editor).  it's not compiled.
Component: Untriaged → Feed Reader
Product: Thunderbird → MailNews Core
ping?
Attached patch feedCache.patchSplinter Review
add check to avoid error needing restart.
Attachment #773308 - Attachment is obsolete: true
Attachment #773308 - Flags: review?(mbanner)
Attachment #773308 - Flags: approval-comm-aurora?
Attachment #788110 - Flags: review?(mbanner)
Attachment #788110 - Flags: approval-comm-beta?
Attachment #788110 - Flags: approval-comm-aurora?
Comment on attachment 788110 [details] [diff] [review]
feedCache.patch

I've just not had time to look at this, maybe Magnus can.
Attachment #788110 - Flags: review?(mbanner) → review?(mkmelin+mozilla)
Comment on attachment 788110 [details] [diff] [review]
feedCache.patch

Review of attachment 788110 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good to me! r=kmelin
Attachment #788110 - Flags: review?(mkmelin+mozilla) → review+
https://hg.mozilla.org/comm-central/rev/d3212436fe26
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 26.0
Comment on attachment 788110 [details] [diff] [review]
feedCache.patch

I'd certainly like to get this out for some testing. a=Standard8
Attachment #788110 - Flags: approval-comm-beta?
Attachment #788110 - Flags: approval-comm-beta+
Attachment #788110 - Flags: approval-comm-aurora?
Attachment #788110 - Flags: approval-comm-aurora+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: