Merge pull request #2762 from derekpierre/no-bytes

Don't put bytes directly into log - instead write as hex

newsfragments/2762.bugfix.rst

@@ -0,0 +1 @@
+Stop writing bytes to log file which causes exceptions - instead write the hex representation.

nucypher/network/nodes.py

@@ -822,7 +822,7 @@ class Learner:
             # TODO: Bucket separately and report.
             unresponsive_nodes.add(current_teacher)  # This does nothing.
             self.known_nodes.mark_as(current_teacher.InvalidNode, current_teacher)
-            self.log.warn(f"Teacher {str(current_teacher)} is invalid: {bytes(current_teacher)}:{e}.")
+            self.log.warn(f"Teacher {str(current_teacher)} is invalid (hex={bytes(current_teacher).hex()}):{e}.")
             self.suspicious_activities_witnessed['vladimirs'].append(current_teacher)
             return
         except RuntimeError as e:
@@ -832,11 +832,13 @@ class Learner:
                 return RELAX
             else:
                 self.log.warn(
-                    f"Unhandled error while learning from {str(current_teacher)}: {bytes(current_teacher)}:{e}.")
+                    f"Unhandled error while learning from {str(current_teacher)} "
+                    f"(hex={bytes(current_teacher).hex()}):{e}.")
                 raise
         except Exception as e:
             self.log.warn(
-                f"Unhandled error while learning from {str(current_teacher)}: {bytes(current_teacher)}:{e}.")  # To track down 2345 / 1698
+                f"Unhandled error while learning from {str(current_teacher)} "
+                f"(hex={bytes(current_teacher).hex()}):{e}.")  # To track down 2345 / 1698
             raise
         finally:
             # Is cycling happening in the right order?