Skip Menu |
 

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Net-Twitter CPAN distribution.

Report information
The Basics
Id: 32244
Status: resolved
Worked: 12 hours (720 min)
Priority: 0/
Queue: Net-Twitter

People
Owner: cthom [...] cpan.org
Requestors: jules [...] zjuul.net
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



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>
Download (untitled) / with headers
text/plain 901b
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
Download patch.txt
text/plain 1.3k

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

From: cthom [...] cpan.org
Download (untitled) / with headers
text/plain 957b
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.
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>
Download (untitled) / with headers
text/plain 1.3k
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.
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>
Download (untitled) / with headers
text/plain 423b
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
I applied this patch, with some tweaks, and it should now function fully in 1.08.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.