entropyxyz / sdk

Official JavaScript SDK for Entropy blockchain.
GNU Affero General Public License v3.0
7 stars 0 forks source link

Sign Function on Web throws / errors #253

Closed benschac closed 8 months ago

benschac commented 8 months ago

client PR: https://github.com/entropyxyz/entropy-client/pull/138

repo:

Screenshot 2023-11-20 at 1 11 38 PM

chachapoly isn't exporting js.

Screenshot 2023-11-20 at 1 12 50 PM

tagging @ameba23 for visibility as well.

ameba23 commented 8 months ago

I'm not sure but i think this comes from use x25519chacha20poly1305's from_hex function with a string which is not valid hex.

The latest version of x25519chacha20poly1305 (v0.2.0) will throw a JS error rather than giving you some cryptic message like this, so try using that to see if that is the problem.

@jawndiego 's branch sandbox_v2 uses that updated version - so try on that branch and see what you get.

benschac commented 8 months ago

Awesome, thanks

Was told to use the sdk via npm. lmk when sandbox_v2 changes are in.

On Tue, Nov 28, 2023 at 11:35 AM peg @.***> wrote:

I'm not sure but i think this comes from use x25519chacha20poly1305's from_hex function with a string which is not valid hex.

The latest version of x25519chacha20poly1305 (v0.2.0) will throw a JS error rather than giving you some cryptic message like this, so try using that to see if that is the problem.

@jawndiego https://github.com/jawndiego 's branch sandbox_v2 https://github.com/entropyxyz/entropy-js/tree/jawn/sandbox_v2 uses that updated version - so try on that branch and see what you get.

— Reply to this email directly, view it on GitHub https://github.com/entropyxyz/entropy-js/issues/253#issuecomment-1830259837, or unsubscribe https://github.com/notifications/unsubscribe-auth/AATDCI6QT2SKIMHYJATB3NDYGYHGDAVCNFSM6AAAAAA7TKQW7SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZQGI2TSOBTG4 . You are receiving this because you authored the thread.Message ID: @.***>

-- Benjamin

frankiebee commented 8 months ago

look for next release. we mentioned in sync on Monday @benschac that its on it's way.