Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Twitter streaming is now exclusively SSL #21

Merged
merged 1 commit into from
Apr 10, 2012
Merged

Conversation

migbar
Copy link
Contributor

@migbar migbar commented Sep 30, 2011

Just changed the default options

@CloudMarc
Copy link

+1 - Need ssl asap please

@carloslopes
Copy link
Contributor

+1 - Please, pull this to the master branch!

@blakejakopovic
Copy link

+1 - Please pull to master

@stve
Copy link
Contributor

stve commented Oct 3, 2011

Until this is merged in, you can force ssl by setting to :ssl => true when connecting:

Twitter::JsonStream.connect(:ssl => true, :auth => ..)

@carloslopes
Copy link
Contributor

yes, we know! But i think that voloko could pull this and the other requests to master

voloko added a commit that referenced this pull request Apr 10, 2012
Twitter streaming is now exclusively SSL
@voloko voloko merged commit 6c00a10 into voloko:master Apr 10, 2012
@voloko
Copy link
Owner

voloko commented Apr 10, 2012

Pulled. BTW this diff was breaking tests: 45fc4ce

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants