Remove more pretty_bad_protocol monkeypatching

This is no longer needed since we don't pass SecureTemporaryFile to
pretty_bad_protocol anymore, it goes to Sequoia instead.

Author: legoktm

securedrop/secure_tempfile.py

@@ -8,7 +8,6 @@
 from cryptography.hazmat.primitives.ciphers import Cipher
 from cryptography.hazmat.primitives.ciphers.algorithms import AES
 from cryptography.hazmat.primitives.ciphers.modes import CTR
-from pretty_bad_protocol._util import _STREAMLIKE_TYPES
 
 
 class SecureTemporaryFile(_TemporaryFileWrapper):
@@ -134,9 +133,3 @@ def close(self) -> None:
         # Since tempfile._TemporaryFileWrapper.close() does other cleanup,
         # (i.e. deleting the temp file on disk), we need to call it also.
         super().close()
-
-
-# python-gnupg will not recognize our SecureTemporaryFile as a stream-like type
-# and will attempt to call encode on it, thinking it's a string-like type. To
-# avoid this we append it the list of stream-like types.
-_STREAMLIKE_TYPES.append(_TemporaryFileWrapper)

securedrop/tests/test_secure_tempfile.py

@@ -1,7 +1,6 @@
 import os
 
 import pytest
-from pretty_bad_protocol._util import _is_stream
 from secure_tempfile import SecureTemporaryFile
 
 MESSAGE = "410,757,864,530"
@@ -78,10 +77,6 @@ def test_file_is_removed_from_disk():
     assert not os.path.exists(f.filepath)
 
 
-def test_SecureTemporaryFile_is_a_STREAMLIKE_TYPE():
-    assert _is_stream(SecureTemporaryFile("/tmp"))
-
-
 def test_buffered_read():
     f = SecureTemporaryFile("/tmp")
     msg = MESSAGE * 1000