search

enarx / ciborium

CBOR utilities
Other
222 stars 54 forks source link

fix: bignum parsed as `Value` #109

Closed ahmedcharles closed 4 months ago

ahmedcharles commented 4 months ago

Fixes: #31

ahmedcharles commented 4 months ago

I added a test which verifies that bignums now get represented by the shorted byte string (without leading zeros).

ahmedcharles commented 4 months ago

Thanks.

rjzak commented 4 months ago

Thank you for your recent PRs, they're a big help to the project. If you're not planning anything soon, maybe I should do a release.

ahmedcharles commented 4 months ago

I just filed #116. I would like to resolve #36 but that will likely require significant changes to the api and I'm thinking about what the best approach there would be. I'll follow up in that issue.

That said, doing a release after that last PR seems like a good idea. Thanks.