Skip to content

Commit ffbc954

Browse files
authored
Merge pull request SocketCluster#24 from MegaGM/10.0.0-breaking-changes
check if middleware does exist
2 parents e79a057 + 54a51d2 commit ffbc954

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

scserver.js

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ var AuthTokenInvalidError = scErrors.AuthTokenInvalidError;
1515
var AuthTokenNotBeforeError = scErrors.AuthTokenNotBeforeError;
1616
var AuthTokenError = scErrors.AuthTokenError;
1717
var SilentMiddlewareBlockedError = scErrors.SilentMiddlewareBlockedError;
18+
var InvalidArgumentsError = scErrors.InvalidArgumentsError;
1819
var InvalidOptionsError = scErrors.InvalidOptionsError;
1920
var InvalidActionError = scErrors.InvalidActionError;
2021
var BrokerError = scErrors.BrokerError;
@@ -639,6 +640,10 @@ SCServer.prototype.generateId = function () {
639640
};
640641

641642
SCServer.prototype.addMiddleware = function (type, middleware) {
643+
if (!this._middleware[type]) {
644+
throw new InvalidArgumentsError(`Middleware type "${type}" is not supported`);
645+
// Read more: https://socketcluster.io/#!/docs/middleware-and-authorization
646+
}
642647
this._middleware[type].push(middleware);
643648
};
644649

0 commit comments

Comments
 (0)