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.

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

Print CBOR_TYPE_BYTESTRING value as hex string #281

Merged
merged 1 commit into from
Jun 3, 2023

Conversation

akallabeth
Copy link
Contributor

Description

Add data as hex string in cbor_describe for type CBOR_TYPE_BYTESTRING

I did find it useful to have the data printed out just like for CBOR_TYPE_STRING.

@PJK
Copy link
Owner

PJK commented Jun 3, 2023

Thank you!

@PJK PJK merged commit 219c2e7 into PJK:master Jun 3, 2023
PJK added a commit that referenced this pull request Jun 3, 2023
@akallabeth akallabeth deleted the bytestring_pretty_print branch June 5, 2023 07:57
netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this pull request Feb 4, 2024
0.11.0 (2024-02-04)
---------------------
- [Updated documentation to refer to RFC 8949](PJK/libcbor#269)
- Improvements to `cbor_describe`
  - [Bytestring data will now be printed as well](PJK/libcbor#281) by  [akallabeth](https://github.com/akallabeth)
  - [Formatting consistency and clarity improvements](PJK/libcbor#285)
- [Fix `cbor_string_set_handle` not setting the codepoint count](PJK/libcbor#286)
- BREAKING: [`cbor_load` will no longer fail on input strings that are well-formed but not valid UTF-8](PJK/libcbor#286)
  - If you were relying on the validation, please check the result using `cbor_string_codepoint_count` instead
- BREAKING: [All decoders like `cbor_load` and `cbor_stream_decode` will accept all well-formed tag values](PJK/libcbor#308) (bug discovered by [dskern-github](https://github.com/dskern-github))
  - Previously, decoding of certain values would fail with `CBOR_ERR_MALFORMATED` or `CBOR_DECODER_ERROR`
  - This also makes decoding symmetrical with serialization, which already accepts all values
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.

2 participants