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

Not finished the backup #1

Open
jyr opened this issue Jul 23, 2014 · 6 comments · May be fixed by #2
Open

Not finished the backup #1

jyr opened this issue Jul 23, 2014 · 6 comments · May be fixed by #2

Comments

@jyr
Copy link

jyr commented Jul 23, 2014

After put the token via web, I have a error:

Fetching 2013-6-14 for room 559196 from campfire
Fetching 2013-6-15 for room 559196 from campfire
Fetching 2013-6-16 for room 559196 from campfire
Fetching 2013-6-17 for room 559196 from campfire

/opt/boxen/nodenv/versions/v0.10.13/lib/node_modules/campfire-archive/lib/CampfireArchiver.js:245
transcript.messages.forEach(this.processTranscriptMessage.bind(this));
^
TypeError: Cannot call method 'forEach' of undefined
at CampfireArchiver.processTranscript (/opt/boxen/nodenv/versions/v0.10.13/lib/node_modules/campfire-archive/lib/CampfireArchiver.js:245:23)
at null. (/opt/boxen/nodenv/versions/v0.10.13/lib/node_modules/campfire-archive/lib/CampfireArchiver.js:238:12)
at /opt/boxen/nodenv/versions/v0.10.13/lib/node_modules/campfire-archive/lib/CampfireArchiver.js:209:13
at Object.oncomplete (fs.js:107:15)

This is it a bug?

@colemanm
Copy link

Just tried this and ran into the same issue. Maybe something to do with the transcript data for the specific day?

@copiousfreetime
Copy link

I just got this, and it has to do with campfire throttling the data pull, from the last archive file pulled down...

% cat 2011-9-11.json
"You have exceeded 50 missing_useragent_header requests in a 10-second period. Please wait 10 seconds before retrying."

Sounds like setting User-Agent header in the request will solve the issue.

@copiousfreetime copiousfreetime linked a pull request Sep 11, 2014 that will close this issue
@crogg
Copy link

crogg commented Jan 7, 2015

This just gets you to the next level of throttling:

$ cat 2014-6-8.json
"You have exceeded 500 web requests in a 10-second period. Please wait 10 seconds before retrying."

@hubertron
Copy link

Getting something similar.

/usr/local/lib/node_modules/campfire-archive/lib/CampfireArchiver.js:245
transcript.messages.forEach(this.processTranscriptMessage.bind(this));
^
TypeError: Cannot read property 'forEach' of undefined
at CampfireArchiver.processTranscript (/usr/local/lib/node_modules/campfire-archive/lib/CampfireArchiver.js:245:22)
at null. (/usr/local/lib/node_modules/campfire-archive/lib/CampfireArchiver.js:238:12)
at /usr/local/lib/node_modules/campfire-archive/lib/CampfireArchiver.js:209:13
at FSReqWrap.oncomplete (fs.js:95:15)

@senny
Copy link

senny commented Jun 8, 2015

Same for me.

@johngb
Copy link

johngb commented Jul 3, 2015

Same problem here

shaochun pushed a commit to shaochun/campfire-archive that referenced this issue Jul 19, 2015
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 a pull request may close this issue.

7 participants