search

superfly / fly

Deploy app servers close to your users. Package your app as a Docker image, and launch it in 17 cities with one simple CLI.
https://fly.io
985 stars 48 forks source link

Added tls options to dev/local runtime. #205

Closed afinch7 closed 4 years ago

afinch7 commented 5 years ago

Added additional options to dev/local server for tls. Just figured I would get the ball rolling for https://github.com/superfly/fly/issues/143.

It should default to a dev certificate and key if none is specified.

I refactored the Server class to be a wrapper for a http server instead of a extension. This is likely to cause some issues, but I don't really see any other good solutions.

mrkurt commented 5 years ago

Nice! Thanks for this. When this is ready I'll make a pre-release npm. I think your changes to server.ts are fine, but we'll need to test it against our live build to make sure it's embedded properly.

afinch7 commented 5 years ago

You can merge this when ready. I've moved on to working on fly.rs for new features.

Spaceghost commented 4 years ago

Is this ready? Unrelated, is fly.rs no more? It got archived today.

mrkurt commented 4 years ago

@Spaceghost we've stopped working on fly.rs in favor of Deno.