Skip to content

Commit

Permalink
pgsql/logger: minor clean-up
Browse files Browse the repository at this point in the history
Remove code for logging fields that should never be logged in the first
place.
  • Loading branch information
jufajardini committed Nov 27, 2023
1 parent 06e8686 commit cdd14c0
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions rust/src/pgsql/logger.rs
Original file line number Diff line number Diff line change
Expand Up @@ -184,12 +184,10 @@ fn log_response(res: &PgsqlBEMessage, jb: &mut JsonBuilder) -> Result<(), JsonEr
}
PgsqlBEMessage::UnknownMessageType(RegularPacket {
identifier: _,
length,
payload,
length: _,
payload: _,
}) => {
// jb.set_string_from_bytes("identifier", identifier.to_vec())?;
jb.set_uint("length", (*length).into())?;
jb.set_string_from_bytes("payload", payload)?;
// We don't want to log these, for now. Cf redmine: #6576
}
PgsqlBEMessage::AuthenticationOk(_)
| PgsqlBEMessage::AuthenticationCleartextPassword(_)
Expand Down

0 comments on commit cdd14c0

Please sign in to comment.