As of January 2014, API is only accessible via SSL endpoints. Twitter redirects all non-SSL visitor traffic to SSL anyway (AFAICT).

We should update all URLs to refer to https://twitter.com.

CommentFileSizeAuthor
#6 twitter-n2239041-6-6.x-5.x.patch18.26 KBdamienmckenna
#5 twitter-n2239041-5-7.x-6.x.patch16.64 KBdamienmckenna
#4 twitter-n2239041-4-7.x-5.x.patch17.09 KBdamienmckenna

Comments

  • Commit 411a0e1 on 8.x-1.x by xurizaemon: 
    Issue #2239041. Restrict Twitter API calls to SSL.
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Active » Fixed
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Fixed » Active
Parent issue: » #2402311: Plan for Twitter v6.x-5.2 release
Related issues: +#2402307: Plan for Twitter v7.x-5.9 release, +#2402309: Plan for Twitter v7.x-6.0 release

No, there are still some URLs with "http" instead of "https".

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Version: 8.x-1.x-dev » 7.x-5.x-dev
Status: Active » Needs review
StatusFileSize
new twitter-n2239041-4-7.x-5.x.patch17.09 KB

This replaces (pretty much) every occurrence of "http://" with "https://" in the 7.x-5.x branch.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Version: 7.x-5.x-dev » 7.x-6.x-dev
StatusFileSize
new twitter-n2239041-5-7.x-6.x.patch16.64 KB

Ported to the 7.x-6.x branch.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Version: 7.x-6.x-dev » 6.x-5.x-dev
StatusFileSize
new twitter-n2239041-6-6.x-5.x.patch18.26 KB

Ported to 6.x-5.x.

The last submitted patch, 5: twitter-n2239041-5-7.x-6.x.patch, failed testing.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Fixed

Committed.

  • DamienMcKenna committed 9248e24 on 7.x-5.x 
    Issue #2239041 by DamienMcKenna: Replaced all http:// URLs with https...

  • DamienMcKenna committed 087c640 on 
    Issue #2239041 by DamienMcKenna: Replaced all http:// URLs with https...

  • DamienMcKenna committed 5dc4819 on 7.x-6.x 
    Issue #2239041 by DamienMcKenna: Replaced all http:// URLs with https...

Status: Fixed » Closed (fixed)

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