search

nellore / vickitrix

Trigger crypto trades on GDAX with the Twitter stream
MIT License
28 stars 7 forks source link

Script placed an order then crashed #6

Closed razorberry closed 7 years ago

razorberry commented 7 years ago

Here is the traceback, the order went through fine.

Traceback (most recent call last):
  File "/usr/local/bin/vickitrix", line 11, in <module>
    sys.exit(go())
  File "/usr/local/lib/python2.7/dist-packages/vickitrix/__init__.py
", line 640, in go
    track=list(keywords)
  File "/usr/local/lib/python2.7/dist-packages/twython/streaming/typ
es.py", line 66, in filter
    self.streamer._request(url, 'POST', params=params)
  File "/usr/local/lib/python2.7/dist-packages/twython/streaming/api
.py", line 154, in _request
    if self.on_success(data):  # pragma: no cover
  File "/usr/local/lib/python2.7/dist-packages/vickitrix/__init__.py
", line 155, in on_success
    or status['user']['screen_name'].lower()
KeyError: 'user'
nellore commented 7 years ago

I'm not sure I completely understand the circumstances around this one, but a fix in https://github.com/nellore/vickitrix/commit/578f26b35f28608a6cd3d89c5b24cfa2b4dd244f will make its way into the next version. Thanks for the bug report!