From 1faa5c47ce5443417fe795662660b4902afa22c4 Mon Sep 17 00:00:00 2001 From: Nick Meves Date: Thu, 1 Jul 2021 19:03:01 -0700 Subject: [PATCH] Remove finicky encryption test AES-CFB is unauthenticated, in rare circumstances it won't error on AES-GCM encrypted payloads --- pkg/apis/sessions/session_state_test.go | 16 ---------------- 1 file changed, 16 deletions(-) diff --git a/pkg/apis/sessions/session_state_test.go b/pkg/apis/sessions/session_state_test.go index e6b9ff39..dccad1af 100644 --- a/pkg/apis/sessions/session_state_test.go +++ b/pkg/apis/sessions/session_state_test.go @@ -264,22 +264,6 @@ func TestEncodeAndDecodeSessionState(t *testing.T) { } }) } - - t.Run("Mixed cipher types cause errors", func(t *testing.T) { - for testName, ss := range testCases { - t.Run(testName, func(t *testing.T) { - cfbEncoded, err := ss.EncodeSessionState(cfb, false) - assert.NoError(t, err) - _, err = DecodeSessionState(cfbEncoded, gcm, false) - assert.Error(t, err) - - gcmEncoded, err := ss.EncodeSessionState(gcm, false) - assert.NoError(t, err) - _, err = DecodeSessionState(gcmEncoded, cfb, false) - assert.Error(t, err) - }) - } - }) }) } }