Skip to content

fix: CID.parse(base36)#307

Merged
lidel merged 1 commit intomasterfrom
fix/cid-parde-base36
Sep 12, 2024
Merged

fix: CID.parse(base36)#307
lidel merged 1 commit intomasterfrom
fix/cid-parde-base36

Conversation

@lidel
Copy link
Copy Markdown
Member

@lidel lidel commented Aug 23, 2024

This PR aims to support base36 out of the box, without having to specify base explicitly.

Rationale is that base36 is used across ecosystem for PeerIDs that have to be represented in base36 to fit in a single DNS label (due to limit of 63 characters).

This aims to support base36 out of the box, without having to specify
the 'base' parameter explicitly.

Rationale is that base36 is used across ecosystem for PeerIDs that have
to be represented in base36 to fit in a single DNS label (due to limit
of 63 characters).
@lidel lidel force-pushed the fix/cid-parde-base36 branch from 2f737ad to daf1874 Compare August 23, 2024 16:41
@lidel lidel requested a review from rvagg August 23, 2024 16:45
@lidel lidel marked this pull request as ready for review August 23, 2024 16:47
@lidel lidel merged commit 892f198 into master Sep 12, 2024
@lidel lidel deleted the fix/cid-parde-base36 branch September 12, 2024 18:49
github-actions bot pushed a commit that referenced this pull request Sep 12, 2024
## [13.2.3](v13.2.2...v13.2.3) (2024-09-12)

### Bug Fixes

* CID.parse(base36) ([#307](#307)) ([892f198](892f198))
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 13.2.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants