Skip to content

Commit b809b05

Browse files
committed
1.0.0
1 parent 4fc974f commit b809b05

File tree

4 files changed

+25
-21
lines changed

4 files changed

+25
-21
lines changed

modules/lib/message.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,8 @@ exports.broadcast = function(socket,data){
2525

2626

2727
exports.sendError = function(socket, message) {
28-
29-
return socket.emit('error', {
28+
console.log('socket.io: send Error to socket client');
29+
socket.emit('error.messages', {
3030
message: message
3131
});
3232

modules/lib/security.js

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,9 @@ const verifyConnection = function(socket, data) {
1919
if(!socket || socket==='undefined' || !clients[socket.id] || !token)
2020
{
2121
console.log('Unauthorized access: access denied');
22-
message.sendError(socket,'Invalid email or password');
22+
message.sendError(socket,'Unauthorized access: access denied');
2323

24-
socket.disconnect();
24+
socket.disconnect(true);
2525
reject(false);
2626
}
2727

@@ -33,23 +33,23 @@ const verifyConnection = function(socket, data) {
3333
if (err)
3434
{
3535

36-
console.log(err);
37-
message.sendError(socket,'Invalid email or password');
36+
console.log('Invalid token');
37+
message.sendError(socket,'Invalid token');
3838

39-
socket.disconnect();
39+
socket.disconnect(true);
4040
reject(false);
4141
}
4242

4343

4444
let userID = decoded.id;
4545

4646
let user = _.find(users,{id:userID});
47-
console.log('founded user',user);
47+
4848
if(!user) {
49-
console.log('!user:false');
50-
message.sendError(socket, 'Invalid email or password');
49+
console.log('User not found');
50+
message.sendError(socket, 'User not found');
5151

52-
socket.disconnect();
52+
socket.disconnect(true);
5353
reject(false);
5454
}
5555
else {
@@ -83,10 +83,10 @@ exports.verifyConnectionAuth0 = function(socket, data) {
8383
if(!socket || socket==='undefined' || !clients[socket.id] || !token)
8484
{
8585
console.log('Unauthorized access: access denied');
86-
message.sendError(socket,'Invalid email or password');
86+
message.sendError(socket,'Unauthorized access: access denied');
8787

88-
socket.disconnect();
89-
reject(false);
88+
socket.disconnect(true);
89+
reject(false);
9090
}
9191

9292
if (token)
@@ -97,9 +97,9 @@ exports.verifyConnectionAuth0 = function(socket, data) {
9797
if (err)
9898
{
9999
console.log(err);
100-
message.sendError(socket,'Invalid email or password');
100+
message.sendError(socket,'Invalid token');
101101

102-
socket.disconnect();
102+
socket.disconnect(true);
103103
reject(false);
104104
}
105105

@@ -111,9 +111,9 @@ exports.verifyConnectionAuth0 = function(socket, data) {
111111
let user = _.find(users,{auth0_user_id:auth0UserID});
112112

113113
if(!user) {
114-
message.sendError(socket, 'Invalid email or password');
114+
message.sendError(socket, 'User not found');
115115

116-
socket.disconnect();
116+
socket.disconnect(true);
117117
reject(false);
118118
}
119119
else {

modules/lib/user.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ exports.getUser = function (socket, data) {
66
security.verifyConnection(socket,data).then(function(result) {
77

88
console.log('Access allowed to getUser()');
9-
if(!data.id) return socket.emit('user.success', null);
9+
if(!data.id) socket.emit('user.success', null);
1010

1111
const user = _.find(users,{id:data.id});
1212

@@ -17,7 +17,7 @@ security.verifyConnection(socket,data).then(function(result) {
1717
id: user.id
1818
} : null;
1919

20-
return socket.emit('user.success', profile);
20+
socket.emit('user.success', profile);
2121

2222
}, function(err) {
2323
console.log('Access denied to getUser()'); return;

modules/mainSocketLocalToken.js

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,23 +28,27 @@ exports.connect = function(server){
2828
io.on('connection', function (socket) {
2929

3030
console.log('socket.io: new connection, current number of authorized connections:',NumberOfConnections());
31+
3132
//Get User
3233
socket.on('getUser', function (data) {
33-
console.log('socket.io: call getUser');
34+
console.log('socket.io: call getUser()');
3435
user.getUser(socket, data);
3536
});
3637

3738
socket.on('message', function (data) {
39+
console.log('socket.io: call message()');
3840
message.broadcast(socket,data);
3941
});
4042

4143
// Login
4244
socket.on('login', function (data) {
45+
console.log('socket.io: call login()');
4346
security.authenticateUser(socket, data);
4447
});
4548

4649
// Logout
4750
socket.on('logout', function (token) {
51+
console.log('socket.io: call logout()');
4852
clients[socket.id] && delete clients[socket.id];
4953
console.log('close connection, number of authorized connections:',NumberOfConnections());
5054

0 commit comments

Comments
 (0)