Skip to content

Commit

Permalink
RCReader: Don't log unknown log types by default
Browse files Browse the repository at this point in the history
There are many known log types we simply don't need to log,
there is no value in enumerating them all or otherwise finding
out about new ones in the server logs.

Also remove remnants of renameuser and makebot log types.
  • Loading branch information
Krinkle committed Oct 13, 2017
1 parent b4cec6f commit 6cfa1c5
Showing 1 changed file with 2 additions and 20 deletions.
22 changes: 2 additions & 20 deletions src/CVNBot/RCReader.cs
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ struct RCEvent
public enum EventType
{
delete, restore, upload, block, unblock, edit, protect, unprotect,
move, rollback, newuser, import, renameuser, makebot, unknown, newuser2, autocreate,
move, rollback, newuser, import, unknown, newuser2, autocreate,
modifyprotect
}

Expand Down Expand Up @@ -413,26 +413,8 @@ void rcirc_OnChannelMessage(object sender, IrcEventArgs e)
//rce.eventtype = RCEvent.EventType.import;
return; //Not interested today
//break;
case "renameuser":
//A user got renamed
//rce.eventtype = RCEvent.EventType.renameuser;
return; //Not interested today
//break;
case "makebot":
//New bot on the block
//rce.eventtype = RCEvent.EventType.makebot;
return; //Not interested today
//break;
case "patrol":
//Marked as patrolled
return; //Not interested today
//break;
case "review":
//FlaggedRevisions or PendingChanges action
return; //Not interested today
//break;
default:
logger.Warn("Unhandled log type: " + logType + " in " + rce.project + ": " + e.Data.Message);
//logger.Warn("Unhandled log type: " + logType + " in " + rce.project + ": " + e.Data.Message);
//Don't react to event
return;
}
Expand Down

0 comments on commit 6cfa1c5

Please sign in to comment.