ryo-ma / deno-websocket

🦕 A simple WebSocket library like ws of node.js library for deno
https://deno.land/x/websocket
MIT License
152 stars 19 forks source link

Works Great, more control on Output #4

Closed khrome83 closed 4 years ago

khrome83 commented 4 years ago

This works really great. I am using it as a backchannel for a development server to build hot reloading.

Would you be open to making a small change, and removing the `console.log("socket connected");

It's minor, but it messes with the log. I would love to have a way to show active or total connections, but I would rather this be an event that is emitted, or status I can fetch and control the output myself.

Thoughts?

ryo-ma commented 4 years ago

Thanks! I think you're right. I have supported that. You should import a new version v0.0.3.

import { WebSocket, WebSocketServer } from "https://deno.land/x/websocket@v0.0.3/mod.ts";
khrome83 commented 4 years ago

awesome!