I don't like the fact that this module transmits user/password in plain text.

HTTPS urls work for Basic auth, why not use them? (or even better, switch to oauth)

Comments

kylehase’s picture

I'm not sure if this is the actual reason but using HTTP guarantees compatibility on any Drupal instance. HTTPS requires that PHP be compiled with SSL (though most are). An option to "use SSL" would be nice but until then, changing http://twitter... to https://twitter... works.

Twitter published the "Security Best Practices" page last month. Where they say

Don't do it. Don't store passwords. Just store OAuth tokens please"
[ http://apiwiki.twitter.com/Security-Best-Practices#PasswordRetention ]

They also recommend using SSL.

abraham’s picture

Status: Active » Fixed

You can use https://twitter.com as the alternative API URL.

Status: Fixed » Closed (fixed)

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

Blooniverse’s picture

Status: Closed (fixed) » Active
Issue tags: +twitter, +ssl, +https, +twitter-ssl, +twitter-api, +twitter-http, +twitter-https

Sorry, I have to reopen this case/issue again.

@abraham, @binford2k: NO, I cannot use https://twitter.com as an alternative API URL (Twitter 6.x-2.5) -- the twitter module won't post anything (resp. it claims to have posted something but on Twitter nothing occurs).

Blooniverse’s picture

... in addition to the above mentioned: when I post from an SSL URL e.g. https://sld-example.tld/edit/node/123, then the Twitter module 'twitters' the SSL URL of this node to/on Twitter.com (for instance as a tiny URL)! I find this very anoying, since I always need to think about changing from https:// to http:// when editing resp. creating a node.

abraham’s picture

Status: Active » Fixed

BasicAuth is being removed from the TwitterAPI in June so if using the SSL URL as an alternative API URL or hardcoding the API URL in the module code don't work for you Have a look at v3 as it supports OAuth. I don't see how the Twitter module handles passwords changing other then to convert to OAuth.

As for comment 5 that is an unrelated issue and should address by creating/contributing to another issue.

Status: Fixed » Closed (fixed)
Issue tags: -twitter, -ssl, -https, -twitter-ssl, -twitter-api, -twitter-http, -twitter-https

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