Removed the need of providing the mode, will be automatically inferre…

…d from the socket presence. The mode it's only used for debug purposes but it's an useful feature to have.

lib/smpp.js

@@ -6,7 +6,7 @@ var net = require('net'),
 	PDU = require('./pdu').PDU,
 	EventEmitter = require('events').EventEmitter;
 
-function Session(mode, options) {
+function Session(options) {
 	EventEmitter.call(this);
 	this.options = options || {};
 	var self = this;
@@ -19,14 +19,16 @@ function Session(mode, options) {
 	this._callbacks = {};
 	this._interval = 0;
 	this._command_length = null;
-	this._mode = mode;
+	this._mode = null;
 	this._id = Math.floor(Math.random() * (999999 - 100000)) + 100000; // random session id
 	this._prevBytesRead = 0;
 	if (options.socket) {
 		// server mode
+		this._mode = "server";
 		this.socket = options.socket;
 	} else {
 		// client mode
+		this._mode = "client";
 		if (options.tls) {
 			transport = tls;
 		}
@@ -242,7 +244,7 @@ function Server(options, listener) {
 	this.options = options;
 
 	transport.Server.call(this, options, function(socket) {
-		var session = new Session("server", {socket: socket, debug: self.options.debug});
+		var session = new Session({socket: socket, debug: self.options.debug});
 		session.debug("client.connected", "client has connected");
 		session.server = self;
 		self.sessions.push(session);
@@ -315,7 +317,7 @@ exports.connect = exports.createSession = function(url, listener) {
 	options.port = options.port || (options.tls ? 3550 : 2775);
 	options.debug = options.debug || false;
 
-	var session = new Session("client", options);
+	var session = new Session(options);
 	if (listener) {
 		session.on(options.tls ? 'secureConnect' : 'connect', listener);
 	}