Get information about a node
requires access to the DB (issue #7)

We should change to using prepared statements where possible as this will keep all the SQL in one place. It also helps with preventing SQL injection.

We also need to handle exceptions (e.g. when bad data is fed to the prepared statements)

currently logs are open closed when we want to log something and we determine the day has changed. This may not be too bad with the server connection logs as there will be regular traffic (ping/pong etc). But for channel/person logs these could hang around for a while - this also means we don't get a useful === close log ==== message until something actively causes the logs to close.

Simple fix would be to have the Handler process detect day changes and go through all log handles calling close on them (requires a new method in the Logger class)

On the first run the log folder isn't created by the server process (only when the handler is called). This leads to lots of errors from the server process not being able to write it's logs

The bot doesn't log messages it's sent itself. This could be due to #12 need to check the IRC docs...

!upload 1164295 would get string from upload.id=1164295

if a change is made to a header file a make clean && make is required otherwise issues (usually segfaults) may occur.

needs new method in Logger class to close the file if it's from a previous day, then get the handler thread to call that method for all files from time to time (once a day ?)

Support something like: !msg craag@confgw1 Hello!

IRC log
http://www.ukhas.net/irclogs/%23ukhasnet-2014-07-15.log

Requires database stuff to be working #7

Try and detect if the bot is being abused

Requires adding in irc::Channel and irc::Member classes as well as tracking of joins/parts . Then add code in to the handler to get the seen timestamp and provide suitable messages (some of which may already be in place).

In
https://github.com/m1ari/UKHASnet-bot/blob/master/irc/connection.cpp#L153-L223

If creating the connection fails in irc::Connection::mainLoop (if sockfd ==0) at any point after the socket is initialised the code can hang as sockfd is set but hasn't actually connected to something.

We should probably clear out sockfd (and possibly some other things) in the various if blocks before the goto statement. Probably using close(sockfd).

We should log when people join and leave the channel and when they change their nick

Will allow a live webbased logviewer (similar to logtail)
Means we can store seen data in the DB rather than periodic dumping to json

in irc/connection.cpp
if ( (r=regcomp(&r_privmsg, ":([^!]+)!([^@]+)@([^ ]+) PRIVMSG (.) :(.)", REG_EXTENDED)) != 0){

example message that failed is (from 2014-05-11)
22:16 < craag> screwed up a few permissions :P

These are similar to PRIVMSG packets but we shouldn't respond

for that post ukhasnet balloon flight lookup the location of the nearest food outlet based on a nodeid

https://developers.google.com/places/documentation/search

Whilst for messages to the channel getDest returns the right thing if the message was from a PRIVMSG or NOTICE it points back to ourself not to where we should reply to.

Simple fix might be to add a replyTo() method that uses similar logic to the logging code in the handler code.

related to this it might be worth adding a reply() method to the message class that can use this code and make the choice of NOTICE/PRIVMSG

From 27/5/2014

14:01 < craag> A !tell command that leaves a message for someone, that's then repeated to them by the bot next time they join is useful
14:02 < craag> eg !tell jcoxon Haven't seen you in a while!
14:02 < craag> Then it'll tell him that next time he logs in

Allow the bot to get updates from a specific node and post updates to the DB
Requires connection to the database Issue #7

[10:11:02.128] RX: :craag!~[email protected] PRIVMSG #ukhasnet :ACTION is philcrump btw ;)
should be something like: * craag is philcrump btw ;)

in db/nodes.cpp Nodes::getNodeLastPacket need to check that lastpacket isn't null

From 23/05/2014

11:32 < tjanos> by the way: it seems, not all log files have the same format. My browser dont want to process for ex. this
file 23ukhasnet-2014-05-22.log as a poor text, it said it is binary file
11:35 < tjanos> The broser put the file in the gedit editor, it seems, it is readable, but I see only thios line with a
non-ascii char:
11:36 < tjanos> [10:51:25.787] * mfa298 is obviously failing at typing todayA [] at the and of this line
11:36 -!- flok420 [~folkert@2001:888:13b3:64:5054:ff:fe87:b394] has quit [Read error: Connection reset by peer]
11:38 < mfa298> the text logs are mostly just there for testing the bot so there might be some issues.
11:39 -!- ibanezmatt13 [~[email protected]] has quit [Quit: TV]
11:39 < mfa298> there's plenty of bug and TODO's in the code where I know there's work to do but it's good enough for most
things.
11:40 < tjanos> it seems me, it is a hex 0x01 and 0x0A chars at the end of your line
11:40 < mfa298> There's a TODO in the source code about that!
11:40 < tjanos> this is another line with this ends: [14:28:38.695] * mfa298 still needs to watch the gps stuffA
11:41 < tjanos> [16:10:38.594] * ibanezmatt13 is now revising, shh ;)A
11:41 < tjanos> seems the lins, started with *
11:42 < tjanos> lines
11:42 < mfa298> It's known about! https://github.com/m1ari/UKHASnet-bot/blob/master/handler.cpp#L70
11:42 < tjanos> Ok, I need to learn a little more

If the bot get's disconnected it stays running but doesn't do anything. Should detect disconnects and attempt to re-join.

This will be fun to test!