Bug #83783 for WebService-Soundcloud: Authentication Issue

Wed Mar 06 07:12:13 2013 jns [...] gellyfish.co.uk - Ticket created

Subject:	Authentication Issue
Date:	Wed, 06 Mar 2013 12:11:23 +0000
To:	bug-WebService-Soundcloud [...] rt.cpan.org
From:	Jonathan Stowe <jns [...] gellyfish.co.uk>

Hi, I found that the released version didn't work for me at all, it appeared that it was making the POST to /oauth2/token with query parameters in the URI rather than as url-encoded content in the body of the request as stated explicitly in the API docs. This resulted in get_authorization_token() always failing. I have made a fork of the code at https://github.com/jonathanstowe/WebService-Soundcloud which contains the fix. I have no intention of releasing this myself but please feel free to merge the changes back. I also have made some other possibly useful changes: * Support credential based authentication for applications with no UI * get_object method returns the decoded response * get_list method that returns a completed collection (e.g. /tracks) as a list, using limit and offset parameters to read the complete collection iteratively. Hope that Helps. -- Rabid Gravy: http://rabidgravy.com/ On Soundcloud: http://soundcloud.com/rabidgravy Twitter: http://twitter.com/RabidGravy

Wed Mar 06 11:14:08 2013 MPGUTTA [...] cpan.org - Correspondence added

Hi, Thanks for reporting it and +1 for providing fix. I will take a look into it and merge changes from https://github.com/jonathanstowe/WebService-Soundcloud. I will do it tomorrow morning my time(GMT +5:30). Thanks in advance. Best Regards, Mohan On Wed Mar 06 07:12:13 2013, JSTOWE wrote: Show quoted text

> Hi, > I found that the released version didn't work for me at all, it

appeared Show quoted text

> that it was making the POST to /oauth2/token with query parameters in > the URI rather than as url-encoded content in the body of the request

as Show quoted text

> stated explicitly in the API docs. This resulted in > get_authorization_token() always failing. > > I have made a fork of the code at > > https://github.com/jonathanstowe/WebService-Soundcloud > > which contains the fix. I have no intention of releasing this myself > but please feel free to merge the changes back. > > I also have made some other possibly useful changes: > > * Support credential based authentication for applications

with Show quoted text

> no UI > * get_object method returns the decoded response > > * get_list method that returns a completed collection > (e.g. /tracks) as a list, using limit and offset parameters to > read the complete collection iteratively. > > Hope that Helps. >

Wed Mar 06 11:14:09 2013 The RT System itself - Status changed from 'new' to 'open'

Wed Mar 06 11:14:09 2013 MPGUTTA [...] cpan.org - Given to MPGUTTA

Wed Mar 06 12:22:43 2013 jns [...] gellyfish.co.uk - Correspondence added

Subject:	Re: [rt.cpan.org #83783] Authentication Issue
Date:	Wed, 06 Mar 2013 17:21:51 +0000
To:	bug-WebService-Soundcloud [...] rt.cpan.org
From:	Jonathan Stowe <jns [...] gellyfish.co.uk>

I'm just fixing the downloads at the moment if you are interested. The issue there is that the request to /tracks/<id>/download results in a redirect to the actual resource on Amazon S3 which contains the S3 Signature required in the URL but by default also has the Authorization header from the original request which doesn't work. Have fun. On Wed, 2013-03-06 at 11:14 -0500, Mohan Prasad Gutta via RT wrote: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=83783 > > > Hi, > > Thanks for reporting it and +1 for providing fix. I will take a look > into it and merge changes from > https://github.com/jonathanstowe/WebService-Soundcloud. > > I will do it tomorrow morning my time(GMT +5:30). > > Thanks in advance. > > Best Regards, > Mohan > > On Wed Mar 06 07:12:13 2013, JSTOWE wrote:

> > Hi, > > I found that the released version didn't work for me at all, it

> appeared

> > that it was making the POST to /oauth2/token with query parameters in > > the URI rather than as url-encoded content in the body of the request

> as

> > stated explicitly in the API docs. This resulted in > > get_authorization_token() always failing. > > > > I have made a fork of the code at > > > > https://github.com/jonathanstowe/WebService-Soundcloud > > > > which contains the fix. I have no intention of releasing this myself > > but please feel free to merge the changes back. > > > > I also have made some other possibly useful changes: > > > > * Support credential based authentication for applications

> with

> > no UI > > * get_object method returns the decoded response > > > > * get_list method that returns a completed collection > > (e.g. /tracks) as a list, using limit and offset parameters to > > read the complete collection iteratively. > > > > Hope that Helps. > >

> >

Sun Mar 10 07:17:33 2013 MPGUTTA [...] cpan.org - Correspondence added 30 min

Hi Jonathan Stowe, Thank you for your bug fixes, pod updates and code cleanup. I have picked required parts from your github version and released them as version 0.02 sometime back to pause server. it will take few hours to be available in CPAN. Thanks again for your time in providing patches. Best Regards, Mohan

Sun Mar 10 07:17:34 2013 MPGUTTA [...] cpan.org - Status changed from 'open' to 'resolved'