feat: export error handle & merge config

Signed-off-by: MiaoWoo <admin@yumc.pw>

packages/websocket/src/server/config.ts

@@ -1,6 +0,0 @@
-import { EventEmitter } from 'events'
-
-export interface NettyWebSocketServerOptions {
-    event: EventEmitter,
-    path?: string;
-}

packages/websocket/src/server/constants.ts

@@ -3,6 +3,7 @@ export enum ServerEvent {
     connect = 'connect',
     connection = 'connection',
     message = 'message',
+    error = 'error',
     disconnect = 'disconnect'
 }
 

packages/websocket/src/server/httprequest.ts

@@ -1,5 +1,6 @@
 import { HttpRequestHandlerAdapter } from '../netty'
-import { Keys, AttributeKeys } from './constants'
+import { AttributeKeys } from './constants'
+import { ServerOptions } from 'socket-io'
 
 const DefaultHttpResponse = Java.type('io.netty.handler.codec.http.DefaultHttpResponse')
 const DefaultFullHttpResponse = Java.type('io.netty.handler.codec.http.DefaultFullHttpResponse')
@@ -14,21 +15,13 @@ const RandomAccessFile = Java.type('java.io.RandomAccessFile')
 const DefaultFileRegion = Java.type('io.netty.channel.DefaultFileRegion')
 const ChannelFutureListener = Java.type('io.netty.channel.ChannelFutureListener')
 
-export type HttpRequestConfig = {
-    root?: string;
-    ws?: string;
-}
-
 export class HttpRequestHandler extends HttpRequestHandlerAdapter {
     private ws: string;
     private root: string;
-    constructor(config: HttpRequestConfig = {
-        root: root + '/wwwroot',
-        ws: '/ws'
-    }) {
+    constructor(options: ServerOptions) {
         super()
-        this.root = config.root;
-        this.ws = config.ws;
+        this.root = options.root;
+        this.ws = options.path;
     }
     channelRead0(ctx: any, request: any) {
         if (request.getUri().startsWith(this.ws)) {

packages/websocket/src/server/index.ts

@@ -4,17 +4,18 @@ import { ServerEvent, Keys } from './constants'
 import { WebSocketDetect } from './websocket_detect'
 import { WebSocketHandler } from './websocket_handler'
 import { NettyClient } from './client'
-import { NettyWebSocketServerOptions } from './config'
+import { ServerOptions, Server } from '../socket-io'
 
 class NettyWebSocketServer extends EventEmitter {
     private pipeline: any;
     private allClients: { [key: string]: NettyClient };
 
-    constructor(pipeline: any, options: NettyWebSocketServerOptions) {
+    constructor(pipeline: any, options: ServerOptions) {
         super()
         this.allClients = {};
         this.pipeline = pipeline;
         let connectEvent = options.event;
+        try { this.pipeline.remove(Keys.Detect) } catch (error) { }
         this.pipeline.addFirst(Keys.Detect, new WebSocketDetect(connectEvent).getHandler())
         connectEvent.on(ServerEvent.detect, (ctx, channel) => {
             channel.pipeline().addFirst(Keys.Handler, new WebSocketHandler(options).getHandler())
@@ -26,8 +27,10 @@ class NettyWebSocketServer extends EventEmitter {
             this.emit(ServerEvent.connect, nettyClient);
         })
         connectEvent.on(ServerEvent.message, (ctx, msg) => {
-            let channel = ctx.channel();
-            this.emit(ServerEvent.message, this.allClients[channel.id()], msg.text())
+            this.emit(ServerEvent.message, this.allClients[ctx.channel().id()], msg.text())
+        })
+        connectEvent.on(ServerEvent.error, (ctx, cause) => {
+            this.emit(ServerEvent.error, this.allClients[ctx.channel().id()], cause)
         })
     }
     close() {

packages/websocket/src/server/text_websocket_frame.ts

@@ -1,11 +1,11 @@
 import { TextWebSocketFrameHandlerAdapter } from '../netty'
 import { EventEmitter } from 'events'
 import { ServerEvent } from './constants'
-import { NettyWebSocketServerOptions } from './config';
+import { ServerOptions } from '../socket-io';
 
 export class TextWebSocketFrameHandler extends TextWebSocketFrameHandlerAdapter {
     private event: EventEmitter;
-    constructor(options: NettyWebSocketServerOptions) {
+    constructor(options: ServerOptions) {
         super()
         this.event = options.event;
     }
@@ -18,6 +18,6 @@ export class TextWebSocketFrameHandler extends TextWebSocketFrameHandlerAdapter
         this.event.emit(ServerEvent.message, ctx, msg)
     }
     exceptionCaught(ctx: any, cause: Error) {
-        console.ex(cause)
+        this.event.emit(ServerEvent.error, ctx, cause)
     }
 }

packages/websocket/src/server/websocket_handler.ts

@@ -4,7 +4,7 @@ import { Keys } from './constants'
 import { WebSocketHandlerAdapter } from "../netty"
 import { HttpRequestHandler } from './httprequest'
 import { TextWebSocketFrameHandler } from './text_websocket_frame'
-import { NettyWebSocketServerOptions } from './config'
+import { ServerOptions } from '../socket-io'
 
 const CharsetUtil = Java.type('io.netty.util.CharsetUtil')
 
@@ -14,8 +14,8 @@ const HttpObjectAggregator = Java.type('io.netty.handler.codec.http.HttpObjectAg
 const WebSocketServerProtocolHandler = Java.type('io.netty.handler.codec.http.websocketx.WebSocketServerProtocolHandler')
 
 export class WebSocketHandler extends WebSocketHandlerAdapter {
-    private options: NettyWebSocketServerOptions;
-    constructor(options: NettyWebSocketServerOptions) {
+    private options: ServerOptions;
+    constructor(options: ServerOptions) {
         super()
         this.options = options;
     }
@@ -32,7 +32,7 @@ export class WebSocketHandler extends WebSocketHandlerAdapter {
             pipeline.addLast('http', new HttpServerCodec())
             pipeline.addLast('chunk', new ChunkedWriteHandler())
             pipeline.addLast('httpobj', new HttpObjectAggregator(64 * 1024))
-            pipeline.addLast('http_request', new HttpRequestHandler().getHandler())
+            pipeline.addLast('http_request', new HttpRequestHandler(this.options).getHandler())
             pipeline.addLast('websocket', new WebSocketServerProtocolHandler(this.options.path, true))
             pipeline.addLast('websocket_handler', new TextWebSocketFrameHandler(this.options).getHandler())
         }

packages/websocket/src/socket-io/index.ts

@@ -1,6 +1,6 @@
 import { EventEmitter } from 'events'
 
-import { NettyWebSocketServer } from '../server'
+import { NettyWebSocketServer, NettyClient } from '../server'
 import { ServerEvent } from '../server/constants';
 
 import { Namespace } from './namespace';
@@ -12,6 +12,17 @@ import { Packet } from './packet';
 import { Socket } from './socket';
 import { Adapter } from './adapter';
 
+interface ServerOptions extends SocketIO.ServerOptions {
+    event?: EventEmitter;
+    root?: string;
+}
+
+const defaultOptions: ServerOptions = {
+    event: new EventEmitter(),
+    path: '/socket.io',
+    root: root + '/wwwroot'
+}
+
 class Server implements SocketIO.Server {
     private nettyServer: NettyWebSocketServer;
     private allClients: { [key: string]: Client };
@@ -24,15 +35,15 @@ class Server implements SocketIO.Server {
     local: SocketIO.Server;
     parser = new Parser();
     _adapter: Adapter;
-    options: SocketIO.ServerOptions;
+    options: ServerOptions;
 
-    constructor(pipeline: any, options: SocketIO.ServerOptions) {
+    constructor(pipeline: any, options: ServerOptions) {
         if (!pipeline) { throw new Error('Netty Pipeline can\'t be undefiend!') }
         this.allClients = {};
         this.nsps = {};
         this.sockets = new Namespace('/', this);
         this.nsps['/'] = this.sockets;
-        this.initNettyServer(pipeline, options);
+        this.initNettyServer(pipeline, Object.assign(defaultOptions, options));
     }
 
     checkRequest(req: any, fn: (err: any, success: boolean) => void): void {
@@ -144,17 +155,18 @@ class Server implements SocketIO.Server {
     };
 
     private initNettyServer(pipeline, options) {
-        this.nettyServer = new NettyWebSocketServer(pipeline, {
-            event: new EventEmitter(),
-            path: options.path
-        });
-        this.nettyServer.on(ServerEvent.connect, (nettyClient) => {
+        this.nettyServer = new NettyWebSocketServer(pipeline, options);
+        this.nettyServer.on(ServerEvent.connect, (nettyClient: NettyClient) => {
             let client = new Client(this, nettyClient);
             this.onconnection(client);
         })
-        this.nettyServer.on(ServerEvent.message, (nettyClient, text) => {
+        this.nettyServer.on(ServerEvent.message, (nettyClient: NettyClient, text) => {
             this.processPacket(this.parser.decode(text), this.allClients[nettyClient.id]);
         })
+        this.nettyServer.on(ServerEvent.error, (nettyClient: NettyClient, cause) => {
+            console.error(`Client ${nettyClient.id} cause error: ` + cause)
+            console.ex(cause)
+        })
     }
 
     private processPacket(packet: Packet, client: Client) {
@@ -193,6 +205,6 @@ class Server implements SocketIO.Server {
 export {
     Server,
     Socket,
-    Server as SocketIOServer,
-    Client as SocketIOClient
+    Client,
+    ServerOptions
 }