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

When (check) intervals are too small, app crashes. #17

Closed
brentlintner opened this issue Mar 10, 2014 · 2 comments
Closed

When (check) intervals are too small, app crashes. #17

brentlintner opened this issue Mar 10, 2014 · 2 comments
Assignees
Labels
Milestone

Comments

@brentlintner
Copy link
Owner

You get stuff like this:

./bin/torrentfish --db foo.db -i .0001 -e .5 -u "http://www.torrentday.com/torrents/rss?l28;l12;l25;l11;l3;l21;l13;l1;l24;l14;l26;l7;l2;u=138040;tp=bd8d5be5da87a0c920adcfde91ce721c"
db info  loaded -> /Users/brent/src/torrentfish/foo.db
poll info  beginning poll
poll info    check interval: 0.0001h
poll info    email interval: 0.5h
poll info  initially checking feeds

...

poll info  added - Bad Country 2014 720p HDRip x264-KingStoner
poll info  added - Media Watch 2014 03 10 PDTV x264-RTA
poll info  added - The Bag Man 2014 BRRip x264 AAC-SSN
poll info  added - Sunday Night 2014 03 09 PDTV x264-RTA
db info  flushing records to disk -> /Users/brent/src/torrentfish/foo.db

TypeError: Cannot call method 'reduce' of null
    at matches_all (/Users/brent/src/torrentfish/lib/daemon/poll.js:11:18)
    at /Users/brent/src/torrentfish/lib/daemon/poll.js:21:11
    at Array.forEach (native)
    at Function.<anonymous> (/Users/brent/src/torrentfish/lib/daemon/poll.js:20:28)
    at process._tickCallback (node.js:415:13)
@brentlintner brentlintner added bug and removed bug labels Mar 10, 2014
@brentlintner
Copy link
Owner Author

Why is the watchlist null?..

Is this a bug, or is this the need for check intervals to be queued (sequentially), vs with setInterval (which is probably not the best thing to do).

@brentlintner brentlintner added this to the v0,4,0 milestone Mar 10, 2014
@brentlintner brentlintner self-assigned this Mar 10, 2014
@brentlintner
Copy link
Owner Author

This was a bug with daemon/poll. Fixing.

brentlintner added a commit that referenced this issue Mar 13, 2014
A lot of other work, refactoring, fixing bugs..

I'm too lazy to sort it out.

Also Fixes GitHub Issues:

- Tests
#15
#18

- Bugs
#11
#17
#14
#16
brentlintner added a commit that referenced this issue Mar 13, 2014
A lot of other work, refactoring, fixing bugs..

I'm too lazy to sort it out.

Also Fixes GitHub Issues:

- Tests
#15
#18

- Bugs
#11
#17
#14
#16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant