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

Add deno port #8

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from
Draft

Add deno port #8

wants to merge 8 commits into from

Conversation

thgh
Copy link

@thgh thgh commented Jun 28, 2020

No ts errors 86 ts errors (down from 200) It's a start!

Anyone that wants to work further on this, go for it!

Biggest blockers are:

  • missing builtin "stream" module for Duplex
  • missing builtin "net" module
  • missing builtin "crypto" module for AES gcm

Tested with: deno run testfile.ts:

import { Node, Context } from '../flatend/deno/src/mod.ts'

await Node.start({
  addrs: ['127.0.0.1:6283'],
  services: {
    hello_world: (ctx: Context) => ctx.send('Hello world!' + Date.now()),
  },
})

@thgh
Copy link
Author

thgh commented Jul 5, 2020

I don't think I can finish this, it probably makes more sense to rewrite the networking part in idiomatic Deno instead of waiting for the std/node/... to catch up.

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.

1 participant