[bug] SocketIONamespace missed method getRoomOperations

[dgolovin-dev]

in spec http://socket.io/docs/rooms-and-namespaces/

Within each namespace, you can also define arbitrary channels that sockets can join and leave.

[dgolovin-dev]

https://github.com/mrniko/netty-socketio/blob/master/src/main/java/com/corundumstudio/socketio/SocketIONamespace.java

[dgolovin-dev]

и ещё, я не уверен, что SocketIOServer.getRoomOperations работает корректно

[dgolovin-dev]

Если смотреть на ориги��альный socket.io-server, то:

https://github.com/Automattic/socket.io/blob/master/lib/index.js#L50
this.sockets = this.of('/');

https://github.com/Automattic/socket.io/blob/master/lib/index.js#L361
['on', 'to', 'in', 'use', 'emit', 'send', 'write'].forEach(function(fn){
Server.prototype[fn] = function(){
var nsp = this.sockets[fn];
return nsp.apply(this.sockets, arguments);
};
});

[mrniko]

в чем проблема?

[dgolovin-dev]

Значит, что отправка в комнату сообщения из сервера должна работать также как из namespace '/', а сейчас, как я вижу по коду собираются комнаты по всем namespace

[dgolovin-dev]

проблема в использовании NamespacesHub.getRoomClients в SocketIOServer.getRoomOperations

[mrniko]

в Namespace.roomClients только клиенты от этого namespace

[mrniko]

получается, что Namespace.getRoomClients работает верно

[dgolovin-dev]

Да, Namespace.getRoomClients работает верно, но в SocketIOServer вот что:

SocketIOServer:

public BroadcastOperations getRoomOperations(String room) {
    Iterable<SocketIOClient> clients = namespacesHub.getRoomClients(room);
    return new BroadcastOperations(clients, configCopy.getStoreFactory());
}

NamespaceHub:

public BroadcastOperations getRoomOperations(String room) {
    Iterable<SocketIOClient> clients = namespacesHub.getRoomClients(room);
    return new BroadcastOperations(clients, configCopy.getStoreFactory());
}

[mrniko]

т.е. ты предлагаешь пределать чтобы SocketIOServer.getRoomOperations работал только с default namespace-ом?

[dgolovin-dev]

Да, оригинальный сервер работает именно так.

[mrniko]

а как по всем namespace-ам рассылать тогда?

[dgolovin-dev]

определить новую операцию, если это имеет смысл... в оригинальном сервере этого нет.
В спецификации явно говорится, что комната привязана к namespace, т.е. для отправки сообщения в комнату нужен namespace, по умолчанию(метод in на сервере) сообщения отсылает в комнату "/" namespace

[pablojr]

Guys, for the rest of the mortals would you consider please posting in English. If not, may I ask you to exchange private e-mails? Thank you very much

[dgolovin-dev]

Russian is international language... ok. Problem in "SocketIOServer.getRoomOperations". Current implementation allows to sent message in target rooms of all namespaces. But in original js socket.io "Server.in" allows to sent message in room of '/' namespace. I think this is bug.