253245 - RSS feeds for Bonsai queries

Status: RESOLVED WONTFIX

(Webtools Graveyard :: Bonsai, enhancement)

Description

[Lukas Petrovicky (CZilla)]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7) Gecko/20040723 Firefox/0.9.1+ (mmoy-Release-092-Pentium-4M)
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7) Gecko/20040723 Firefox/0.9.1+ (mmoy-Release-092-Pentium-4M)

I often check the given link for updates. it would make my life much easier, if
only there were RSS or ATOM, that would give me the information I seek.

I know that many RSS readers take a lot of bandwidth, but I think it'd be worth it.

Reproducible: Always
Steps to Reproduce:
1.
2.
3.

Actual Results:  
there is no RSS or ATOM feed for Bonsai queries

Expected Results:  
there should be some RSS or ATOM feed for Bonsai queries

Comment 1

[Myk Melez [:myk] [@mykmelez]]

See also bug 262967.

Comment 2

[Henrik Gemal]

this would be very nice. Perhaps if I have the time I'd look into this.

Comment 3

[Mike Taylor [:bear]]

I will work up a patch against the latest bonsai tree to do this.

Comment 4

[Chris Lawson (gone)]

Ping.

Any progress on this yet?

I'm especially interested in what ideas people might have for what <title> should be. I also think it would be very useful if links to the changed files were included. Basically, I'd like to be able to do everything I can do from a bonsai query results page without actually loading that page in my browser.

cl

Comment 5

[Mike Taylor [:bear]]

Attachment: first rough patch - work in progress

This is a very rough draft (have I said that enough  ;)

it adds the feed option and generates a basic atom formated output.  It needs human readable dates and id's

Comment 6

[Phil Ringnalda (:philor)]

Comment on attachment 266114 [details] [diff] [review]
first rough patch - work in progress

Let the feed pedantry begin!

>+} elsif ($show_feed) {
>+    print "Content-type: text/xml\n\n";

application/atom+xml, please, so you don't depend on browser sniffing heuristics

>+        print "    <author>$ci_who</author>\n"

<author> has one required child element, <name>, and optional <url> and <email> children (though I assume <name> will be a name only a CVS admin could love, it's still what passes for a name).

Comment 7

[Mike Taylor [:bear]]

Attachment: generates somewhat valid atom data

this version actually works (I used it in my test bonsai) and it generates almost valid atom.

the only part that should be added are links and id's

Comment 8

[Mike Taylor [:bear]]

thanks phil - I just saw your notes as I was posting the latest.  let me make those changes now.

(In reply to comment #6)
> (From update of attachment 266114 [details] [diff] [review])
> Let the feed pedantry begin!
> 
> >+} elsif ($show_feed) {
> >+    print "Content-type: text/xml\n\n";
> 
> application/atom+xml, please, so you don't depend on browser sniffing
> heuristics
> 
> >+        print "    <author>$ci_who</author>\n"
> 
> <author> has one required child element, <name>, and optional <url> and <email>
> children (though I assume <name> will be a name only a CVS admin could love,
> it's still what passes for a name).
> 

Comment 9

[Mike Taylor [:bear]]

Attachment: slightly more valid atom - includes phil's suggestions

Comment 10

[Dave Miller [:justdave]]

The <summary> construct doesn't appear to be displayed at all by NetNewsWatcher, which is a really popular RSS client on Mac.  And that's the main part of the commit that I'd be interested in.  Maybe that should be in the <content> before the list of files?

This patch is live on http://bonsai-stage.mozilla.org/ if anyone wants to play.

Comment 11

[Henrik Gemal]

We should also add link rel header.

Test link for those who just wants to click:
http://bonsai-stage.mozilla.org/cvsquery.cgi?treeid=default&module=all&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=day&mindate=&maxdate=&cvsroot=%2Fcvsroot&feed=1

Comment 12

[Tuukka Tolvanen (sp3000)]

feedvalidator has a few issues with the current stage version: http://feedvalidator.org/check.cgi?url=http%3A%2F%2Fbonsai-stage.mozilla.org%2Fcvsquery.cgi%3Fdate%3Dday%26feed%3D1

Comment 13

[Mike Taylor [:bear]]

Attachment: added link rel=self; fixed update dates and moved more of the information into content

Comment 14

[Henrik Gemal]

I still think that the HTML version should have a header like:
<link rel="alternate" type="application/atom+xml" title="Atom" href="blabla.org" />

Comment 15

[Dave Miller [:justdave]]

(In reply to comment #10)
> This patch is live on http://bonsai-stage.mozilla.org/ if anyone wants to play.

And no longer, because it's not in CVS yet, and it conflicts with the recently-added graphing stuff.

Comment 16

[Adam Muntner [:adamm] (use NEEDINFO)]

Bonsai was decommissioned, closing all remaining bugs "wontfix"