From 8aa5b91f24cf82b89b075db9eecdfb62967d8f12 Mon Sep 17 00:00:00 2001 From: Savely Krendelhoff Date: Sat, 23 Aug 2025 13:08:06 +0700 Subject: [PATCH] [PHASE-7] Remove validation from message decoder --- internal/protocol/message_decoder.go | 15 --------------- internal/protocol/message_decoder_test.go | 10 ---------- 2 files changed, 25 deletions(-) diff --git a/internal/protocol/message_decoder.go b/internal/protocol/message_decoder.go index 72eea1f..f22441e 100644 --- a/internal/protocol/message_decoder.go +++ b/internal/protocol/message_decoder.go @@ -25,11 +25,6 @@ func (d *MessageDecoder) Decode(r io.Reader) (*Message, error) { return nil, fmt.Errorf("failed to read message type: %w", err) } - // Validate message type (only request types are valid for server) - if !isValidRequestType(msgType) { - return nil, fmt.Errorf("invalid message type: 0x%02x", msgType) - } - // Read payload length (4 bytes, big-endian) var payloadLength uint32 if err := binary.Read(r, binary.BigEndian, &payloadLength); err != nil { @@ -53,13 +48,3 @@ func (d *MessageDecoder) Decode(r io.Reader) (*Message, error) { PayloadStream: payloadStream, }, nil } - -// isValidRequestType checks if the message type is a valid request type -func isValidRequestType(msgType MessageType) bool { - switch msgType { - case ChallengeRequestType, SolutionRequestType: - return true - default: - return false - } -} diff --git a/internal/protocol/message_decoder_test.go b/internal/protocol/message_decoder_test.go index 9b8cf47..1acc8c1 100644 --- a/internal/protocol/message_decoder_test.go +++ b/internal/protocol/message_decoder_test.go @@ -70,16 +70,6 @@ func TestMessageDecoder_Decode_Errors(t *testing.T) { data: []byte{}, wantErr: "EOF", }, - { - name: "invalid message type", - data: []byte{0xFF, 0x00, 0x00, 0x00, 0x00}, - wantErr: "invalid message type", - }, - { - name: "response type not allowed", - data: []byte{0x02, 0x00, 0x00, 0x00, 0x00}, // ChallengeResponseType - wantErr: "invalid message type", - }, { name: "incomplete header", data: []byte{0x01, 0x00, 0x00},