Skip to content

Loading…

change remaining http:// to https:// #32

Merged
merged 1 commit into from

2 participants

@eddelbuettel

This seems to get bti past the initial hurdle.

@eddelbuettel eddelbuettel referenced this pull request
Closed

Change at Twitter? #31

@gregkh
Owner

Doh, that was simple, so sorry for not doing that sooner...

@gregkh gregkh merged commit 0620029 into gregkh:master
@eddelbuettel

Yes, in hindsight we all look silly. But it would have helped if twitter had a new machine / address available for testing. By having the old behavior back for a few days we had no means to really test.

But all good now -- so thanks as always for bti and your continued paternal oversight.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Jan 15, 2014
  1. @eddelbuettel
Showing with 2 additions and 2 deletions.
  1. +2 −2 bti.c
View
4 bti.c
@@ -258,9 +258,9 @@ static void bti_curl_buffer_free(struct bti_curl_buffer *buffer)
free(buffer);
}
-const char twitter_host[] = "http://api.twitter.com/1.1/statuses";
+const char twitter_host[] = "https://api.twitter.com/1.1/statuses";
const char twitter_host_stream[] = "https://stream.twitter.com/1.1/statuses"; /*this is not reset, and doesnt work */
-const char twitter_host_simple[] = "http://api.twitter.com/1.1";
+const char twitter_host_simple[] = "https://api.twitter.com/1.1";
const char twitter_name[] = "twitter";
static const char twitter_request_token_uri[] = "https://twitter.com/oauth/request_token";
Something went wrong with that request. Please try again.