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

feat: byo piece hasher #1323

Merged
merged 1 commit into from
Mar 19, 2024
Merged

feat: byo piece hasher #1323

merged 1 commit into from
Mar 19, 2024

Conversation

alanshaw
Copy link
Member

@alanshaw alanshaw commented Mar 4, 2024

Allows users to bring their own piece hasher. i.e. they can use the faster fr32-sha2-256-trunc254-padded-binary-tree-multihash hasher if their environment allows them to load the WASM.

Switches the client to use the JS implementation by default.

refs #1322

Copy link
Contributor

@vasco-santos vasco-santos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@alanshaw alanshaw merged commit fe2e3d5 into main Mar 19, 2024
14 checks passed
@alanshaw alanshaw deleted the feat/byo-piece-hasher branch March 19, 2024 14:52
vasco-santos added a commit that referenced this pull request Mar 21, 2024
🤖 I have created a release *beep* *boop*
---


##
[13.1.0](upload-client-v13.0.1...upload-client-v13.1.0)
(2024-03-21)


### Features

* byo piece hasher
([#1323](#1323))
([fe2e3d5](fe2e3d5))
* non-parallel piece hashing and CAR upload
([#1305](#1305))
([7a6385b](7a6385b))
* upgrade ucanto/transport to 9.1.0 in all packages to get more verbose
errors from HTTP transport on non-ok response
([#1312](#1312))
([d6978d7](d6978d7))


### Fixes

* upload client should perform filecoin offer
([#1333](#1333))
([466e3f7](466e3f7))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: Vasco Santos <[email protected]>
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