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

CommentFileSizeAuthor
#7 1814610-proper-api-urls-6.x-3.x.patch1.77 KBfizk
#1 1814610-proper-api-urls.patch1.44 KBfizk

Comments

fizk’s picture

fizk commented
StatusFileSize
new 1814610-proper-api-urls.patch1.44 KB

Here's a patch.

fizk’s picture

fizk commented
Priority: Normal » Major
fizk’s picture

fizk commented
Status: Active » Needs review
fizk’s picture

fizk commented
Issue summary: View changes

Clarify description.

tbenice’s picture

tbenice commented

Patch #1 worked. Thanks.

ytsuhako’s picture

ytsuhako commented

Patch #1 worked me too. thank you!

juampynr’s picture

juampynr commented
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

fizk commented
StatusFileSize
new 1814610-proper-api-urls-6.x-3.x.patch1.77 KB

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

joestewart’s picture

joestewart commented
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

scottrouse commented

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

xurizaemon’s picture

xurizaemon
he/him
Location Ōtepoti, Aotearoa 🏝
commented
Status: Reviewed & tested by the community » Fixed

Committed in 118d859

juampynr’s picture

juampynr commented

Many thanks @fizk!

fizk’s picture

fizk commented

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

lyricnz’s picture

lyricnz commented

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

xurizaemon’s picture

xurizaemon
he/him
Location Ōtepoti, Aotearoa 🏝
commented
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

fizk commented

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

xurizaemon’s picture

xurizaemon
he/him
Location Ōtepoti, Aotearoa 🏝
commented

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

(not verified) commented
Issue summary: View changes

Link to other discussion