(In reply to Magnus Melin from comment #4)
> Comment on attachment 754287
> patch
> 
> Review of attachment 754287:
> -----------------------------------------------------------------
> 
> Suite's newsblog.properties needs updating too.
> 
> (Seems this patch needs bug 254231 applied first.)
> 

it shouldn't be necessary; some minor checks/logs that happened to be
there because i did it first were moved here, to make that one simple
for sec.

> 100 sounds like a fairly small limit, how about 500 or something? i think
> i've seen valid feeds have way above 100
> 

how about 200?  even for 100 it will take 10+ seconds to complete
(unlike news, content is downloaded not just headers).  it's not so much
legitimacy as informing the user what they're about to get into, so they
can do it later etc.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/191006

Title:
  Thunderbird RSS should have a per feed limit per download

To manage notifications about this bug go to:
https://bugs.launchpad.net/thunderbird/+bug/191006/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to