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

Master #16

Open
wants to merge 60 commits into
base: master
Choose a base branch
from
Open

Master #16

wants to merge 60 commits into from

Conversation

chanaoka
Copy link

Lock all of Close instead of just conn.stale

Aaron Raddon and others added 30 commits December 8, 2011 18:24
…on serialzation and consumption to handlers, add metrics
This is defined in twittertypes.go

Other 'id' types are int64, so maybe this is an oversight?

This caused the error 'json: cannot unmarshal number into Go value of type string' when unmarshalling JSON returned by the Twitter REST API. The error was fixed by changing the type.

BTW, twitter also returns a 'in_reply_to_status_id_str' value if a string is needed, although this value is not set in this file.
Tweet.In_reply_to_status_id type is string and should be int64.
Felix Jodoin and others added 30 commits April 12, 2013 17:25
Additional parameter to add location support.
Adding locations as parameter to filter (breaking api change)
Fix data race detected by go's race detector
Ran across this problem when running another project's test suite with
-race.
Lock all of Close instead of just conn.stale
implement support for insecure https connections, return error when getting a connection without response
Add timeouts on TLSHandshake and ResponseHeader
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.

10 participants