Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[BUG] MessageDef logging of ByteString does not use fixed-width hex numbers #22706

Closed
bzbarsky-apple opened this issue Sep 16, 2022 · 0 comments · Fixed by #22707
Closed

[BUG] MessageDef logging of ByteString does not use fixed-width hex numbers #22706

bzbarsky-apple opened this issue Sep 16, 2022 · 0 comments · Fixed by #22707

Comments

@bzbarsky-apple
Copy link
Contributor

bzbarsky-apple commented Sep 16, 2022 •
edited
Loading

Reproduction steps

You get things like

0xac, 0x30, 0xa, 0x6, 0x8, 0x2a, 0x86,

instead of

0xac, 0x30, 0x0a, 0x06, 0x08, 0x2a, 0x86,

which is a lot easier to convert into a hex byte sequence with no "0x" in the middle.

Bug prevalence

Always

GitHub hash of the SDK that was being used

69551a1

Platform

core

Platform Version(s)

No response

Anything else?

No response
bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this issue Sep 16, 2022
@bzbarsky-apple
Make MessageDef byte string logging nicer.
8950048 
0-pad the values, so they are constant-width.

Fixes project-chip#22706
@bzbarsky-apple bzbarsky-apple mentioned this issue Sep 16, 2022
Merged
andy31415 pushed a commit that referenced this issue Sep 19, 2022
@bzbarsky-apple
Make MessageDef byte string logging nicer. (#22707)
21b708e 
0-pad the values, so they are constant-width.

Fixes #22706
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Make MessageDef byte string logging nicer. bzbarsky-apple/connectedhomeip
1 participant
@bzbarsky-apple