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

Store message logs for each channel #6

Closed
zarino opened this issue May 4, 2015 · 2 comments · Fixed by #20
Closed

Store message logs for each channel #6

zarino opened this issue May 4, 2015 · 2 comments · Fixed by #20

Comments

@zarino
Copy link
Owner

zarino commented May 4, 2015

Eg: irc.server.net/#somechannel/2015-05-04.txt

@zarino
Copy link
Owner Author

zarino commented Jul 10, 2015

Something like

var path = app.getPath('userData') + '/' + serverName + '/' + channelName + '/' + isoDate + '.txt';
fs.appendFile(path, theStringToBeLogged, function (err) {
  // callback
});

Looks like it's easiest to have the server-side write this file, but if that's the case, messages will come from the IRC daemon, to the server, to the client for parsing and display, and then back to the server for logging. Maybe the server should do all the parsing of the IRC daemon's output, and just send tidy data to the client for templating?

Also, I'm a bit worried that the dates/times might not match between server and client. Really each message should be given a timestamp of when it was received by the IRC daemon (or as close as possible) and then that should be used everywhere.

@zarino
Copy link
Owner Author

zarino commented Jul 10, 2015

Log format for messages:

[2015-07-10T11:54:21Z] <zarino> hello world

Actions:

[2015-07-10T11:54:16Z] • zarino: files a github ticket

Stage directions:

[2015-07-10T11:59:36Z] zarino left the channel

Topic:

[2015-07-10T09:24:49Z] Topic is Hello World!

zarino added a commit that referenced this issue Jul 10, 2015
@zarino zarino mentioned this issue Jul 10, 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.

1 participant