Skip to content

twitter input plugin fixes #1450

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 4 commits into from
Closed

twitter input plugin fixes #1450

wants to merge 4 commits into from

Conversation

@colinsurprenant
Copy link
Contributor

@colinsurprenant colinsurprenant commented Jun 18, 2014

  • all tweet hash keys should be strings, not symbols
  • add robustness, rescue exceptions, handle Twitter rate limiting errors

@colinsurprenant colinsurprenant mentioned this pull request Jul 23, 2014
Closed
@colinsurprenant colinsurprenant added this to the v1.5.0 milestone Jul 23, 2014
@colinsurprenant colinsurprenant self-assigned this Jul 23, 2014
@colinsurprenant colinsurprenant mentioned this pull request Aug 13, 2014
Closed
@elasticsearch-release
Copy link

elasticsearch-release commented Aug 26, 2014

💚 Test passed.
Refer to this link for build results: http://build-eu-1.elasticsearch.org/job/Logstash_PR/81/

colinsurprenant added a commit that referenced this pull request Sep 27, 2014
@colinsurprenant @jordansissel
add robustness to tweets stream handling
10ac156 
Fixes #1450
colinsurprenant added a commit that referenced this pull request Sep 27, 2014
@colinsurprenant @jordansissel
monkey patch twitter gem to trap json parsing exceptions and use our …
a78f6ba 
…own json parser

Fixes #1450
colinsurprenant added a commit that referenced this pull request Sep 27, 2014
@colinsurprenant @jordansissel
support :symbolize_keys options for json parsing
d2f6118 
Fixes #1450
colinsurprenant added a commit that referenced this pull request Sep 27, 2014
@colinsurprenant @jordansissel
stringify all symbols in twitter hash
603ad12 
Fixes #1450
colinsurprenant added a commit that referenced this pull request Sep 27, 2014
@colinsurprenant @jordansissel
add robustness to tweets stream handling
3092dee 
Fixes #1450
colinsurprenant added a commit that referenced this pull request Sep 27, 2014
@colinsurprenant @jordansissel
monkey patch twitter gem to trap json parsing exceptions and use our …
61970d9 
…own json parser

Fixes #1450
colinsurprenant added a commit that referenced this pull request Sep 27, 2014
@colinsurprenant @jordansissel
support :symbolize_keys options for json parsing
4d7f893 
Fixes #1450
@jordansissel jordansissel added v1.x and removed O(2) labels Sep 27, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@colinsurprenant colinsurprenant

Labels

bug

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@colinsurprenant @elasticsearch-release @jordansissel