pablobarbera / instaR

Dev version of instaR package: Access to Instagram API via R
106 stars 47 forks source link

#34 correction to instaOAuth.R #35

Closed j0bby closed 8 years ago

j0bby commented 8 years ago

changed testing URL (found in API's documentation) for recent httr version : removed '/' into HTTR_SERVER_PORT as it is showed in hadley/httr/R/oauth-listener.r

pablobarbera commented 8 years ago

Thanks a lot for the PR, much appreciated! I ended up implementing a slightly different version of this fix, but same idea. :-)

j0bby commented 8 years ago

hum yes, my commit changed the behaviour for versions less than 1.2, did not think about it ^^ But still there is an issue with the URL you use to test the authentification is successful ... I tried it with your new commit and it doesn't work ... I replaced it with this one and it worked great: https://api.instagram.com/v1/users/self/?

pablobarbera commented 8 years ago

Oops, true! Sorry I missed that. You're absolutely right. I just fixed this in 905907c14b6b122fdae5fff58a01ab789dda2914 Thanks again!