Cloudflare Worker support

[lukasjaronis]

Can we add support for CF workers? Would need to use node:crypto instead of crypto.

[dantio]

Yep, definitely planed. However, it's more work than just replace the 'crypto' with 'node: crypto' ...

[lukasjaronis]

Only thing that was getting in the way of using the ebayApi in a Cloudflare Worker was the digitalSignature, which I locally patched to account for environment and use the appropriate crypto API. XML parsing is not an issue since you can set compatibility flags in a worker to make it work.

[dantio]

@lukasjaronis
Might you share all your changes? That would be great.

[cjlawson02]

Here's my diff:

diff --git a/node_modules/ebay-api/dist/api/digitalSignature.js b/node_modules/ebay-api/dist/api/digitalSignature.js
index 6f544dc..566e884 100644
--- a/node_modules/ebay-api/dist/api/digitalSignature.js
+++ b/node_modules/ebay-api/dist/api/digitalSignature.js
@@ -1,4 +1,4 @@
-import { createHash, sign } from 'crypto';
+import { createHash } from 'node:crypto';
 import { EBayError } from '../errors/index.js';
 const beginPrivateKey = '-----BEGIN PRIVATE KEY-----';
 const endPrivateKey = '-----END PRIVATE KEY-----';
@@ -60,7 +60,7 @@ export function generateSignature(headers, privateKey, signatureComponents, payl
     if (!privateKey.startsWith(beginPrivateKey)) {
         privateKey = beginPrivateKey + '\n' + privateKey + '\n' + endPrivateKey;
     }
-    const signatureBuffer = sign(undefined, Buffer.from(baseString), privateKey);
+    const signatureBuffer = crypto.subtle.sign(undefined, privateKey, Buffer.from(baseString));
     const signature = signatureBuffer.toString('base64');
     return `sig1=:${signature}:`;
 }