Log unknown TCP packets from driver station.

field/arena.go

@@ -649,7 +649,7 @@ func (arena *Arena) Update() {
 	msSinceLastDsPacket := int(time.Since(arena.lastDsPacketTime).Seconds() * 1000)
 	if sendDsPacket || msSinceLastDsPacket >= dsPacketPeriodMs {
 		if msSinceLastDsPacket >= dsPacketWarningMs && arena.lastDsPacketTime.After(time.Time{}) {
-			log.Printf("Too long since last driver station packet: %dms", msSinceLastDsPacket)
+			log.Printf("Warning: Long time since last driver station packet: %dms", msSinceLastDsPacket)
 		}
 		arena.sendDsPacket(auto, enabled)
 		arena.ArenaStatusNotifier.Notify()
@@ -683,7 +683,7 @@ func (arena *Arena) Run() {
 			go arena.runPeriodicTasks()
 		}
 		if time.Since(loopStartTime).Microseconds() > arenaLoopWarningUs {
-			log.Printf("Arena loop iteration took too long: %dus", time.Since(loopStartTime).Microseconds())
+			log.Printf("Warning: Arena loop iteration took a long time: %dus", time.Since(loopStartTime).Microseconds())
 		}
 
 		time.Sleep(time.Millisecond * arenaLoopPeriodMs)

field/driver_station_connection.go

@@ -371,19 +371,22 @@ func (dsConn *DriverStationConnection) handleTcpConnection(arena *Arena) {
 
 		packetType := int(buffer[2])
 		switch packetType {
-		case 28:
+		case 29:
 			// DS keepalive packet; do nothing.
+			continue
 		case 22:
 			// Robot status packet.
 			var statusPacket [36]byte
 			copy(statusPacket[:], buffer[2:38])
 			dsConn.decodeStatusPacket(statusPacket)
-		}
 
-		// Log the packet if the match is in progress.
-		matchTimeSec := arena.MatchTimeSec()
-		if matchTimeSec > 0 && dsConn.log != nil {
-			dsConn.log.LogDsPacket(matchTimeSec, packetType, dsConn)
+			// Create a log entry if the match is in progress.
+			matchTimeSec := arena.MatchTimeSec()
+			if matchTimeSec > 0 && dsConn.log != nil {
+				dsConn.log.LogDsPacket(matchTimeSec, packetType, dsConn)
+			}
+		default:
+			log.Printf("Received unknown packet type %d from Team %d", packetType, dsConn.TeamId)
 		}
 	}
 }