The API URLs for most Twitter operations must be versioned, eg. https://api.twitter.com/1.1/statuses/update.json

However, the API URLs for oAuth operations must not versioned, eg. https://api.twitter.com/oauth/authenticate

This has also been discussed in #1809720: On cron: Unexpected error: Sorry, that page does not exist

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

fizk’s picture

Here's a patch.

fizk’s picture

Priority: Normal » Major
fizk’s picture

Status: Active » Needs review
fizk’s picture

Issue summary: View changes

Clarify description.

tbenice’s picture

Patch #1 worked. Thanks.

ytsuhako’s picture

Patch #1 worked me too. thank you!

juampynr’s picture

Version: 7.x-3.x-dev » 6.x-3.x-dev

Committed. Many thanks.

http://drupalcode.org/project/twitter.git/commitdiff/67814a1

¿Any chance to backport this for Drupal 6?

fizk’s picture

Patch backported to 6.x-3.x-dev

joestewart’s picture

Status: Needs review » Reviewed & tested by the community

@juampy - the patch in #7 applies cleanly to 6.x-3.x and corrects the issue.

scottrouse’s picture

Agreed. Patch in #7 works for 6.x-3.x-dev.

xurizaemon’s picture

Status: Reviewed & tested by the community » Fixed

Committed in 118d859

juampynr’s picture

Many thanks @fizk!

fizk’s picture

@juampy, @grobot, Thanks :) Could you also take a look at #1814642: Invalid handling of exceptions causes fatal errors on PHP5.2

lyricnz’s picture

Suggest create a release including this fix, as twitter API is broken without it.

xurizaemon’s picture

Version: 6.x-3.x-dev » 7.x-3.x-dev

IMO this should go forward into 7.x-4.x also, but if #1569982: Integrate with Libraries API is retained then doing so is outside the scope of this module.

fizk’s picture

Whatever the outcome of #1569982: Integrate with Libraries API is, I think the fixes in this issue should be released immediately.

xurizaemon’s picture

Agreed. Happy to make this happen today. Issue for release co-ordination: #1820138: Release co-ordination for Twitter 7.x-3.3 & 6.x-3.0

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

Anonymous’s picture

Issue summary: View changes

Link to other discussion