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

pass batch down to sign and verify function #413

Closed
wants to merge 3 commits into from
Closed

Conversation

chm-diederichs
Copy link
Contributor

No description provided.

@chm-diederichs chm-diederichs force-pushed the batch-auth branch 3 times, most recently from bbb71c6 to 58edb1e Compare August 11, 2023 13:04
@LuKks
Copy link
Contributor

LuKks commented Aug 11, 2023

Corestore will stop working due the breaking changes, should we make it backward compatible or update Corestore accordingly?

test/auth.js Outdated Show resolved Hide resolved
@chm-diederichs
Copy link
Contributor Author

Corestore will stop working due the breaking changes, should we make it backward compatible or update Corestore accordingly?

Updated the PR to be backwards compat, now batch / hash are passed as the last args

@LuKks
Copy link
Contributor

LuKks commented Aug 16, 2023

Cool, last thing I think we can safely remove this unused helper now

exports.treeSignableLegacy = function (hash, length, fork) {

@chm-diederichs
Copy link
Contributor Author

These changes we need were merged in #418.

Now this PR would just be removing the legacy option, which wasn't the original intention, so I'll close this.

@chm-diederichs chm-diederichs deleted the batch-auth branch August 17, 2023 12:44
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