Clarified a confusing field name in JSON logs

GoSecure · May 31, 2024 · 7f766be · 7f766be
1 parent e98ddec
commit 7f766be
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/CHANGELOG.adoc b/CHANGELOG.adoc
@@ -11,6 +11,7 @@ For a detailed view of what has changed, refer to the {uri-repo}/commits/master[
 === Enhancements
 
 * Handles unknown NTSTATUS gracefully ({uri-issue}471[#471])
+* JSON logs field change: data becomes exceptionData
 
 
 == v2.1.0 - 2024-01-23

diff --git a/pyrdp/layer/tcp.py b/pyrdp/layer/tcp.py
@@ -104,7 +104,7 @@ def dataReceived(self, data: bytes):
  if isinstance(e, ParsingError):
  self.log.error("Parser information: %(parserInfo)s", {"parserInfo": e.formatLayer(len(e.layers) - 1)})
 
- self.log.error("Exception occurred when receiving: %(data)s" , {"data": hexlify(data).decode()})
+ self.log.error("Exception occurred when receiving: %(exceptionData)s" , {"exceptionData": hexlify(data).decode()})
 
  raise
 

diff --git a/pyrdp/parser/rdp/fastpath.py b/pyrdp/parser/rdp/fastpath.py
@@ -90,7 +90,7 @@ def parseEvents(self, data: bytes) -> List[FastPathEvent]:
  except KeyboardInterrupt:
  raise
  except Exception:
- log.error("Exception occurred when receiving: %(data)s", {"data": hexlify(eventData)})
+ log.error("Exception occurred when receiving: %(exceptionData)s", {"exceptionData": hexlify(eventData)})
  raise
 
  events.append(event)