earthstar-project/earthstar-streaming-rpc

Like JSON-RPC, but supports streaming.

earthstar-streaming-rpc's Issues

RPC: how to import express in Deno (or use opine?)

Help set up the project so it imports express in node, but opine in deno. Hopefully they are similar enough that our basic usage of them will work in both cases.

Nothing is importing this yet, it will be part of TransportHttpServer when that is written.

Resolve HTTP not working for both Browser and Node in NPM package.

What's the problem you want solved?

When the NPM package is bundled, fetch is shimmed using node-fetch. This shim does not work in the browser, which means that users of the NPM versions of earthstar packages cannot sync via HTTP.

Is there a solution you'd like to recommend?

I need to find some solution for either:

• Shimming with an 'isomorphic' version of fetch which works in both Node and the Browser. I'd probably need to make this myself as there are no packages right now which satisfy dnt's ESM and typings requirements.
• Possibly splitting build process up so that there is both a Node and Browser version of earthstar-streaming-rpc, further complicating things.

"EventTarget is not defined" on node 14

What's the problem you want solved?

npm test fails on node 14:

ReferenceError: EventTarget is not defined
    at Object.<anonymous> (/Users/me/projects/earthstar-streaming-rpc/npm/node_modules/@deno/shim-deno/dist/deno/stable/classes/PermissionStatus.js:7:32)

This is coming from denoland/node_deno_shims also known as @deno/shim-deno, used by dnt.

This was fixed already in deno shims but the fix has not been released, it came a few days after the latest release of 0.1.2.

Is there a solution you'd like to recommend?

We need to encourage shim-deno to make a new npm release.

RPC: HTTP

write

• TransportHttpClient
• TransportHttpServer

RPC: write Readme docs

RPC: websockets

RPC: write tests

RPC: get tests running in various browsers

Make the tests run in a browser somehow. Ideally multiple specific browsers (Chrome, Firefox).

Possibly solutions: browser-run, Jest, Puppeteer, Playwright

RPC: BroadcastChannel

Allow HTTP transport servers to use path opt if present

What's the problem you want solved?

transport-http-server-opine (and -express) currently hijack all of the routes of the opine/express app that is passed into the constructor

Is there a solution you'd like to recommend?

Use the path opt if it exists and if not default to the wildcard