Remove server from WebsocketRouter (#17238)
App contains server. Server contains WebsocketRouter. There is no need for WebsocketRouter to contain server too. As is evident because the code never used it. ```release-note NONE ```
Этот коммит содержится в:
коммит произвёл
GitHub
родитель
017e14d4b5
Коммит
f865703ce9
@@ -496,10 +496,9 @@ func NewServer(options ...Option) (*Server, error) {
|
|||||||
}
|
}
|
||||||
|
|
||||||
s.WebSocketRouter = &WebSocketRouter{
|
s.WebSocketRouter = &WebSocketRouter{
|
||||||
server: s,
|
|
||||||
handlers: make(map[string]webSocketHandler),
|
handlers: make(map[string]webSocketHandler),
|
||||||
|
app: fakeApp,
|
||||||
}
|
}
|
||||||
s.WebSocketRouter.app = fakeApp
|
|
||||||
|
|
||||||
mailConfig := s.MailServiceConfig()
|
mailConfig := s.MailServiceConfig()
|
||||||
|
|
||||||
|
|||||||
@@ -16,7 +16,6 @@ type webSocketHandler interface {
|
|||||||
}
|
}
|
||||||
|
|
||||||
type WebSocketRouter struct {
|
type WebSocketRouter struct {
|
||||||
server *Server
|
|
||||||
app *App
|
app *App
|
||||||
handlers map[string]webSocketHandler
|
handlers map[string]webSocketHandler
|
||||||
}
|
}
|
||||||
|
|||||||
Ссылка в новой задаче
Block a user