Skip to content

Commit

Permalink
removes logs from pings to test cpu hasten
Browse files Browse the repository at this point in the history
  • Loading branch information
“Caio committed May 23, 2022
1 parent 49221bc commit 22bd16f
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions internal/api/handlers/rooms_handler.go
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,6 @@ func (h *RoomsHandler) ForwardPlayerEvent(ctx context.Context, message *api.Forw
func (h *RoomsHandler) UpdateRoomWithPing(ctx context.Context, message *api.UpdateRoomWithPingRequest) (*api.UpdateRoomWithPingResponse, error) {
handlerLogger := h.logger.With(zap.String(logs.LogFieldSchedulerName, message.SchedulerName), zap.String(logs.LogFieldRoomID, message.RoomName))
gameRoom, err := requestadapters.FromApiUpdateRoomRequestToEntity(message)
handlerLogger.Info("handling room ping request", zap.Any("message", message))
if err != nil {
handlerLogger.Error("error parsing ping request", zap.Any("ping", message), zap.Error(err))
return nil, status.Error(codes.InvalidArgument, err.Error())
Expand All @@ -105,8 +104,6 @@ func (h *RoomsHandler) UpdateRoomWithPing(ctx context.Context, message *api.Upda
// TODO(gabrielcorado): should we fail when the status transition fails?
return nil, status.Error(codes.Unknown, err.Error())
}

handlerLogger.Info("Room updated with ping successfully")
return &api.UpdateRoomWithPingResponse{Success: true}, nil
}

Expand Down

0 comments on commit 22bd16f

Please sign in to comment.