Skip to content

Now prints hex representation of non-printable keys when logging them #80

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 11 commits into from
Jan 14, 2025

Conversation

larsewi
Copy link
Owner

@larsewi larsewi commented Jan 14, 2025

  • Fixed confusing log message in JSON API
  • Added tip for how to run acceptance test
  • Fixed log message typo in LCH_BlockLoad
  • Fixed log msg typo in LCH_BufferPrintFormat
  • Fixed typo in LCH_DEFAULT_PREFERRED_CHAIN_LENGTH macro
  • Fixed typo in LCH_InstanceGetPreferredChainLength
  • Fixed typo in log msg in LCH_TableInfoLoad
  • Fixed typo doc string for LCH_DoubleToSize
  • Added function LCH_BufferIsPrintable
  • Added function LCH_BufferToPrintable
  • Now converts keys to printable strings before logging them

Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
It returns true if all characters are of character code 32 to 127 (not
included).

Ticket: CFE-4370
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Function to convert a buffer into a printable string. If the buffer
contains non-printable characters, then a string such as `b"<CONTENT>"`
is returned. If the buffer contains only printable characters, then a
string such as `"<CONTENT>"` is returned.

Ticket: CFE-4370
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
Ticket: CFE-4370
Changelog: Title
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
@larsewi larsewi merged commit 5851acc into master Jan 14, 2025
4 checks passed
@larsewi larsewi deleted the logmsg branch January 20, 2025 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant