Drop native node Buffer API dependency about siwe

[tfalencar]

I created a fork of apg-js, and added safer-buffer to the project. Then I pushed it as a new packaged to npm: buffer-apg-js. I tested in my application and finally the application doesn't crash anymore and I'm able to use siwe in the browser with Vite / SvelteKit. If you'd like to test it as well you can replace the 'siwe' dependency with '@crowdtainer/siwe' (another package I pushed to npm which is a fork of siwe replacing apg-js with buffer-apg-js in the siwe-parser package). I'd like to eventually deprecate my custom fork though (@crowdtainer/siwe), so would be happy to see this solved in this repo (either by using the buffer-apg-js fork, or if spruce decides to create its own npm package with a fork.

I'm no longer using this approach - and not maintaining this package.

[hbriese]

@tfalencar's issue has been closed as unfortunately apg-js is unwilling to find an alternative to Buffer.

Removing dependence on Buffer is incredibly important for portability

[niconiahi]

hey! I'm needing this as well. It's important to understand that many applications (everytime gets bigger) are running on the edge runtime, meaning that we don't have NodeAPIs. To me, it's a must for an application to remove all dependencies from Node and replace them with Web APIs

can at least the mantainer hint what changes would this include? I could take a look at it. The codebase seems small enough to work on

[w4ll3]

@tfalencar Thanks for your contribution, I'll bring up the discussion with the team today, we'll probably have our own fork as you mentioned but we'll make sure to credit your contribution. Will update the results soon.

[jwcheon]

@dankcellar will you open source this in the future? would love to use it!

[twtaylor]

yeah this doesn't work in a FE for some functions

[au2001]

Any update on this?

[bradwbradw]

I created a fork of apg-js, and added safer-buffer to the project. Then I pushed it as a new packaged to npm: buffer-apg-js. I tested in my application and finally the application doesn't crash anymore and I'm able to use siwe in the browser with Vite / SvelteKit.

If you'd like to test it as well you can replace the 'siwe' dependency with '@crowdtainer/siwe' (another package I pushed to npm which is a fork of siwe replacing apg-js with buffer-apg-js in the siwe-parser package).

I'd like to eventually deprecate my custom fork though (@crowdtainer/siwe), so would be happy to see this solved in this repo (either by using the buffer-apg-js fork, or if spruce decides to create its own npm package with a fork.

I tried importing your fork in vite using import { SiweMessage } from '@crowdtainer/siwe' but i get "failed to resolve entry for package "@crowdtainer/siwe""

[tfalencar]

I tried importing your fork in vite using import { SiweMessage } from '@crowdtainer/siwe' but i get "failed to resolve entry for package "@crowdtainer/siwe""

Please check with the Spruce team, it seems they have another library now (SSX), maybe it doesn't have this issue. I don't know if this problem will be solved in spruce/siwe.

[bradwbradw]

hey @tfalencar I would consider SSX but i'm doing a hackathon and I only have a couple days and I need siwe :) I'm not sure if your Buffer-friendly clone can be installed / imported using vite JS, but if you have any tips, I'd really appreciate it. I promise I won't leave it in a prod environment, it's just for a proof of concept

[dankcellar]

For anyone seeing this in 2024, I got it working: https://www.npmjs.com/package/cloudflare-siwe
import { SiweMessage, generateNonce } from 'cloudflare-siwe';