Bug #32244 for Net-Twitter: [patch] Net::Twitter config parameter "source"

Fri Jan 11 06:16:53 2008 jules [...] zjuul.net - Ticket created

Subject:	[patch] Net::Twitter config parameter "source"
Date:	Fri, 11 Jan 2008 12:16:25 +0100
To:	bug-Net-Twitter [...] rt.cpan.org
From:	Jules <jules [...] zjuul.net>

Hello Chris, First of all: thanks for making Net::Twitter! It has made making my own twitter shell rather easy. I've made a patch (see attachment) to set the "source" name of the client. This is needed to set the "from [source]" in stead of "from web". Code that can be used: $t = $twit = Net::Twitter->new( source=>'twitsh', username=>$username, password=>$password); Result (from a dump) -- HTTP headers look like this: Content-Type: application/x-www-form-urlencoded X-Twitter-Client-URL:: http://x4.net/twitter/meta.xml X-Twitter-Client-Version:: 1.06 X-Twitter-Client:: Perl Net::Twitter status=I+am+testing.&source=twitsh My client can be found at http://web.zjuul.net/twitsh (note: this does not contain the above code). I'd appreciate it if you can include this functionality after the review of my patch. Thanks, Jules

Message body is not shown because sender requested not to inline it.

Fri Jan 11 15:04:05 2008 cthom [...] cpan.org - Correspondence added

From:

cthom [...] cpan.org

On Fri Jan 11 06:16:53 2008, jules@zjuul.net wrote: Show quoted text

> I've made a patch (see attachment) to set the "source" name of the > client. This is needed to set the "from [source]" in stead of "from > web".

While I've been aware of this "feature" for some time, I've avoided putting this in in the past because it requires that you petition twitter for your own source to be added to the list. I took your patch and applied it to my own HEAD dev tree for what will eventually become 1.07. It prevents update from working AT ALL. This could be some odd interaction with my post-1.06 code, I'm not sure. I don't have an approved "from" registered with twitter, perhaps that's why. I'll dig into this, hopefully soon. If I can't make it degrade nicely for those without codes, I can't include it. I'm sure I can come up with something. Maybe I need to register a Net::Twitter source with Twitter. I'm out of town all weekend, so I'll look at this early next week.

Fri Jan 11 15:04:08 2008 The RT System itself - Status changed from 'new' to 'open'

Sat Jan 12 05:14:50 2008 jules [...] zjuul.net - Correspondence added

Subject:	Re: [rt.cpan.org #32244] [patch] Net::Twitter config parameter "source"
Date:	Sat, 12 Jan 2008 11:17:44 +0100
To:	bug-Net-Twitter [...] rt.cpan.org
From:	Jules Stuifbergen <jules [...] zjuul.net>

Odd. Maybe it has something to do with Twitter. My updates still say "from web", although Alex (from twitter) said yesterday it should be working "later this day". My updates are still coming through. No hurries though, it isn't really that important, it just adds to the cool factor :-) Have a good weekend, thanks for the reply Jules Chris Thompson via RT schreef: Show quoted text

> <URL: http://rt.cpan.org/Ticket/Display.html?id=32244 > > > On Fri Jan 11 06:16:53 2008, jules@zjuul.net wrote:

>> I've made a patch (see attachment) to set the "source" name of the >> client. This is needed to set the "from [source]" in stead of "from >> web".

> > While I've been aware of this "feature" for some time, I've avoided > putting this in in the past because it requires that you petition > twitter for your own source to be added to the list. > > I took your patch and applied it to my own HEAD dev tree for what will > eventually become 1.07. It prevents update from working AT ALL. This > could be some odd interaction with my post-1.06 code, I'm not sure. I > don't have an approved "from" registered with twitter, perhaps that's why. > > I'll dig into this, hopefully soon. If I can't make it degrade nicely > for those without codes, I can't include it. I'm sure I can come up with > something. Maybe I need to register a Net::Twitter source with Twitter. > > I'm out of town all weekend, so I'll look at this early next week.

Thu Jan 17 09:32:55 2008 jules [...] zjuul.net - Correspondence added

Subject:	Re: [rt.cpan.org #32244] [patch] Net::Twitter config parameter "source"
Date:	Thu, 17 Jan 2008 15:32:34 +0100
To:	Chris Thompson via RT <bug-Net-Twitter [...] rt.cpan.org>
From:	Jules <jules [...] zjuul.net>

Chris, an update on my previous mail: I think the twitter API has undergone some back-end changes. My patched version works now, "suddenly", I'm getting "from twitsh" when using it like: $twit = Net::Twitter->new( source=>'twitsh', username=>$username, password=>$password); $twit->update("bla bla"); and "from web", when I leave out the source=>'twitsh'. Jules

Tue Feb 26 11:53:48 2008 cthom [...] cpan.org - Correspondence added 720 min

I applied this patch, with some tweaks, and it should now function fully in 1.08.

Tue Feb 26 11:53:50 2008 cthom [...] cpan.org - Status changed from 'open' to 'resolved'

Tue Feb 26 11:53:51 2008 cthom [...] cpan.org - Given to CTHOM

Bug #32244 for Net-Twitter: [patch] Net::Twitter config parameter "source"

Preferred bug tracker