rservicebus 0.1.74 → 0.1.75

data/lib/rservicebus/ConfigureMonitor.rb

@@ -44,29 +44,29 @@ module RServiceBus
 
             env.each do |k,v|
                 if v.is_a?(String) and
-					k.start_with?( "RSBOB_" ) then
+					k.start_with?('RSBOB_') then
                     uri = URI.parse( v )
-                    name = k.sub( "RSBOB_", "" )
+                    name = k.sub( 'RSBOB_', '')
                     monitor = nil?
                     case uri.scheme
-                        when "csvdir"
-                        require "rservicebus/Monitor/CsvDir"
+                        when 'csvdir'
+                        require 'rservicebus/Monitor/CsvDir'
 						monitor = Monitor_CsvDir.new( @host, name, uri )
 
-                        when "xmldir"
-                        require "rservicebus/Monitor/XmlDir"
+                        when 'xmldir'
+                        require 'rservicebus/Monitor/XmlDir'
 						monitor = Monitor_XmlDir.new( @host, name, uri )
 
-                        when "dir"
-                        require "rservicebus/Monitor/Dir"
+                        when 'dir'
+                        require 'rservicebus/Monitor/Dir'
 						monitor = Monitor_Dir.new( @host, name, uri )
                         
-                        when "dirnotifier"
-                        require "rservicebus/Monitor/DirNotifier"
+                        when 'dirnotifier'
+                        require 'rservicebus/Monitor/DirNotifier'
 						monitor = Monitor_DirNotifier.new( @host, name, uri )
 
-                        when "csvperlinedir"
-                        require "rservicebus/Monitor/CsvPerLine"
+                        when 'csvperlinedir'
+                        require 'rservicebus/Monitor/CsvPerLine'
 						monitor = Monitor_CsvPerLineDir.new( @host, name, uri )
                         else
 						abort("Scheme, #{uri.scheme}, not recognised when configuring Monitor, #{k}=#{v}");

data/lib/rservicebus/ConfigureSubscriptionStorage.rb

@@ -1,6 +1,6 @@
 module RServiceBus
     
-    require "uri"
+    require 'uri'
     
     #Configure SubscriptionStorage for an rservicebus host
     class ConfigureSubscriptionStorage
@@ -9,12 +9,12 @@ module RServiceBus
             uri = URI.parse( uri_string )
             
             case uri.scheme
-                when "redis"
-                require "rservicebus/SubscriptionStorage/Redis"
+                when 'redis'
+                require 'rservicebus/SubscriptionStorage/Redis'
                 s = SubscriptionStorage_Redis.new( appName, uri )
                 
-                when "file"
-                require "rservicebus/SubscriptionStorage/File"
+                when 'file'
+                require 'rservicebus/SubscriptionStorage/File'
                 s = SubscriptionStorage_File.new( appName, uri )
                 
                 else

data/lib/rservicebus/CronManager.rb

@@ -47,7 +47,7 @@ class CronManager
             hash['cron'] = CronParser.new(cron_string)
             hash['next'] = hash['cron'].next(Time.now)
             @list << hash
-            @Bus.log( "Cron set for, #{n}, #{cron_string}" )
+            @Bus.log( "Cron set for, #{n}, #{cron_string}, next run, #{hash['next']}" )
         end
     end
     
@@ -55,16 +55,16 @@ class CronManager
         @Bus = host
         @msgNames = msgNames
         
-        RServiceBus.rlog "Load Cron"
+        RServiceBus.rlog 'Load Cron'
         @list = Array.new
         ENV.each do |k,v|
-            if k.start_with?( "RSBCRON_" ) then
-                self.addCron( k.sub( "RSBCRON_", "" ), v )
-                elsif k.start_with?( "RSBCRON" ) then
-                v.split( ";" ).each do |v|
-                    parts = v.split( " ", 6 )
+            if k.start_with?('RSBCRON_') then
+                self.addCron( k.sub( 'RSBCRON_', ''), v )
+                elsif k.start_with?('RSBCRON') then
+                v.split(';').each do |v|
+                    parts = v.split( ' ', 6 )
                     
-                    self.addCron( parts.pop, parts.join( " " ) )
+                    self.addCron( parts.pop, parts.join(' ') )
                 end
             end
             

data/lib/rservicebus/EndpointMapping.rb

@@ -17,7 +17,7 @@ module RServiceBus
         def configureMapping( mapping )
             match = mapping.match( /(.+):(.+)/ )
             if match.nil? then
-                log "Mapping string provided is invalid"
+                log 'Mapping string provided is invalid'
                 log "The entire mapping string is: #{mapping}"
                 log "*** Could not find ':' in mapping entry, #{line}"
                 exit()
@@ -28,29 +28,29 @@ module RServiceBus
             
             @queueNameList.each do |q|
                 if q != match[2] && q.downcase == match[2].downcase then
-                    log( "*** Two queues specified with only case sensitive difference." )
+                    log('*** Two queues specified with only case sensitive difference.')
                     log( "*** #{q} != #{match[2]}" )
-                    log( "*** If you meant these queues to be the same, please match case and restart the bus." )
+                    log('*** If you meant these queues to be the same, please match case and restart the bus.')
                 end
             end
             @queueNameList << match[2]
         end
         
         def Configure( localQueueName=nil )
-            self.log( "EndpointMapping.Configure" )
+            self.log('EndpointMapping.Configure')
             
             @queueNameList = []
             @queueNameList << localQueueName unless localQueueName.nil?
-            
-            if !self.getValue( "MESSAGE_ENDPOINT_MAPPING" ).nil? then
-                log( "*** MESSAGE_ENDPOINT_MAPPING environment variable was detected")
-                log( "*** You may have intended MESSAGE_ENDPOINT_MAPPINGS, note the 'S' on the end")
+
+            unless self.getValue('MESSAGE_ENDPOINT_MAPPING').nil? then
+              log('*** MESSAGE_ENDPOINT_MAPPING environment variable was detected')
+              log("*** You may have intended MESSAGE_ENDPOINT_MAPPINGS, note the 'S' on the end")
             end
             
-            mappings = self.getValue( "MESSAGE_ENDPOINT_MAPPINGS" )
+            mappings = self.getValue('MESSAGE_ENDPOINT_MAPPINGS')
             return self if mappings.nil?
             
-            mappings.split( ";" ).each do |mapping|
+            mappings.split(';').each do |mapping|
                 self.configureMapping( mapping )
             end
             
@@ -70,7 +70,7 @@ module RServiceBus
         end
         
         def getSubscriptionEndpoints
-            return @endpoints.keys.select { |el| el.end_with?( "Event" ) }
+            return @endpoints.keys.select { |el| el.end_with?('Event') }
         end
     end
     

data/lib/rservicebus/HandlerLoader.rb

@@ -37,15 +37,15 @@ class HandlerLoader
 #
 # @param [String] filePath the path to be cleaned
 	def getRequirePath( filePath )
-		if !filePath.start_with?( "/" ) then
-			filePath = "./" + filePath
-		end
+		unless filePath.start_with?('/') then
+      filePath = './' + filePath
+    end
 		
 		if File.exists?( filePath ) then
-			return filePath.sub( ".rb", "")
+			return filePath.sub( '.rb', '')
 		end		
 
-		abort( "Filepath, " + filePath + ", given for MessageHandler require doesn't exist" );
+		abort( 'Filepath, ' + filePath + ", given for MessageHandler require doesn't exist" );
 	end
 
 # Instantiate the handler named in handlerName from the file name in filePath
@@ -63,9 +63,9 @@ class HandlerLoader
 		begin
 			handler = Object.const_get(handlerName).new();
 		rescue Exception => e
-			puts "Expected class name: " + handlerName + ", not found after require: " +  requirePath
-			puts "**** Check in " + filePath + " that the class is named : " + handlerName
-			puts "( In case its not that )"
+			puts 'Expected class name: ' + handlerName + ', not found after require: ' +  requirePath
+			puts '**** Check in ' + filePath + ' that the class is named : ' + handlerName
+			puts '( In case its not that )'
 			raise e
 		end
 		
@@ -84,17 +84,17 @@ class HandlerLoader
         end
 
 		begin
-			RServiceBus.rlog "filePath: " + filePath
-			RServiceBus.rlog "handlerName: " + handlerName
+			RServiceBus.rlog 'filePath: ' + filePath
+			RServiceBus.rlog 'handlerName: ' + handlerName
 
 			handler = self.loadHandlerFromFile( handlerName, filePath )
-			RServiceBus.log "Loaded Handler: " + handlerName
+			RServiceBus.log 'Loaded Handler: ' + handlerName
 
             @handlerManager.addHandler( msgName, handler )
             
             @listOfLoadedPaths[filePath] = 1
 		rescue Exception => e
-			puts "Exception loading handler from file: " + filePath
+			puts 'Exception loading handler from file: ' + filePath
 			puts e.message
 			puts e.backtrace[0]
 
@@ -108,7 +108,7 @@ class HandlerLoader
 # @param [String] path directory to check
 # @return [Array] a list of paths to files found in the given path
 	def getListOfFilesForDir( path )
-        list = Dir[path + "/*"];
+        list = Dir[path + '/*'];
         RServiceBus.rlog "HandlerLoader.getListOfFilesForDir. path: #{path}, list: #{list}"
         
         return list
@@ -122,17 +122,17 @@ class HandlerLoader
 # @param [String] baseDir directory to check for handlers of the given msgName
 	def loadHandlersFromSecondLevelPath(msgName, baseDir)
 		self.getListOfFilesForDir(baseDir).each do |filePath|
-			if !filePath.end_with?( "." ) then
-				extName = File.extname( filePath )
-				if !File.directory?( filePath ) &&
-						extName == ".rb" then
+      unless filePath.end_with?('.') then
+        extName = File.extname(filePath)
+        if !File.directory?(filePath) &&
+            extName == '.rb' then
 
-					fileName = File.basename( filePath ).sub( ".rb", "" )
-					handlerName = "MessageHandler_#{msgName}_#{fileName}"
+          fileName = File.basename(filePath).sub('.rb', '')
+          handlerName = "MessageHandler_#{msgName}_#{fileName}"
 
-					self.loadHandler( msgName, filePath, handlerName )
-				end
-			end
+          self.loadHandler(msgName, filePath, handlerName)
+        end
+      end
 		end
 
 		return self
@@ -145,7 +145,7 @@ class HandlerLoader
 	def getMsgName( filePath )
 		baseName = File.basename( filePath )
 		extName = File.extname( baseName )
-		fileName = baseName.sub( extName, "" )
+		fileName = baseName.sub( extName, '')
 
 		msgName = fileName
 		
@@ -158,16 +158,16 @@ class HandlerLoader
 	def loadHandlersFromTopLevelPath(baseDir)
         RServiceBus.rlog "HandlerLoader.loadHandlersFromTopLevelPath. baseDir: #{baseDir}"
 		self.getListOfFilesForDir(baseDir).each do |filePath|
-			if !filePath.end_with?( "." ) then
-
-				msgName = self.getMsgName( filePath )
-				if File.directory?( filePath ) then
-					self.loadHandlersFromSecondLevelPath( msgName, filePath )
-				else
-					handlerName = "MessageHandler_#{msgName}"
-					self.loadHandler( msgName, filePath, handlerName )
-				end
-			end
+      unless filePath.end_with?('.') then
+
+        msgName = self.getMsgName(filePath)
+        if File.directory?(filePath) then
+          self.loadHandlersFromSecondLevelPath(msgName, filePath)
+        else
+          handlerName = "MessageHandler_#{msgName}"
+          self.loadHandler(msgName, filePath, handlerName)
+        end
+      end
 		end
 
 		return self

data/lib/rservicebus/HandlerManager.rb

@@ -26,7 +26,7 @@ module RServiceBus
         def setBusAttributeIfRequested( handler )
             if defined?( handler.Bus ) then
                 handler.Bus = @host
-                RServiceBus.log "Bus attribute set for: " + handler.class.name
+                RServiceBus.log 'Bus attribute set for: ' + handler.class.name
             end
             
             return self
@@ -38,7 +38,7 @@ module RServiceBus
         def setStateAttributeIfRequested( handler )
             if defined?( handler.State ) then
                 handler.State = @stateManager.Get( handler )
-                RServiceBus.log "Bus attribute set for: " + handler.class.name
+                RServiceBus.log 'Bus attribute set for: ' + handler.class.name
             end
             
             return self

data/lib/rservicebus/Host.rb

@@ -68,9 +68,9 @@ module RServiceBus
         #Thin veneer for Configuring state
         #
         def configureSagaStorage
-            string = RServiceBus.getValue( "SAGA_URI" )
+            string = RServiceBus.getValue('SAGA_URI')
             if string.nil? then
-                string = "dir:///tmp"
+                string = 'dir:///tmp'
             end
 
             uri = URI.parse( string )
@@ -104,7 +104,7 @@ module RServiceBus
         #Subscriptions are specified by adding events to the
         #msg endpoint mapping
         def sendSubscriptions
-            log "Send Subscriptions"
+            log 'Send Subscriptions'
 
             @endpointMapping.getSubscriptionEndpoints.each { |eventName| self.Subscribe( eventName ) }
 
@@ -114,7 +114,7 @@ module RServiceBus
         #Load and configure Message Handlers
         #
         def loadHandlers()
-            log "Load Message Handlers"
+            log 'Load Message Handlers'
             @handlerManager = HandlerManager.new( self, @resourceManager, @stateManager )
             @handlerLoader = HandlerLoader.new( self, @handlerManager )
 
@@ -127,7 +127,7 @@ module RServiceBus
 
         #Load and configure Sagas
         def loadSagas()
-            log "Load Sagas"
+            log 'Load Sagas'
             @sagaManager = Saga_Manager.new( self, @resourceManager, @sagaStorage )
             @sagaLoader = SagaLoader.new( self, @sagaManager )
 
@@ -148,7 +148,7 @@ module RServiceBus
         #Load Contracts
         #
         def loadContracts()
-            log "Load Contracts"
+            log 'Load Contracts'
 
             @config.contractList.each do |path|
                 require path
@@ -161,7 +161,7 @@ module RServiceBus
         #For each directory given, find and load all librarys
         #
         def loadLibs()
-            log "Load Libs"
+            log 'Load Libs'
 
             @config.libList.each do |path|
                 $:.unshift path
@@ -224,15 +224,15 @@ module RServiceBus
         #Ignition
         #
         def run
-            log "Starting the Host"
+            log 'Starting the Host'
 
             log "Watching, #{@mq.localQueueName}"
             $0 = "rservicebus - #{@mq.localQueueName}"
-            if !@config.forwardReceivedMessagesTo.nil? then
-                log "Forwarding all received messages to: " + @config.forwardReceivedMessagesTo.to_s
+            unless @config.forwardReceivedMessagesTo.nil? then
+              log 'Forwarding all received messages to: ' + @config.forwardReceivedMessagesTo.to_s
             end
-            if !@config.forwardSentMessagesTo.nil? then
-                log "Forwarding all sent messages to: " + @config.forwardSentMessagesTo.to_s
+            unless @config.forwardSentMessagesTo.nil? then
+              log 'Forwarding all sent messages to: ' + @config.forwardSentMessagesTo.to_s
             end
 
             self.StartListeningToEndpoints
@@ -241,7 +241,7 @@ module RServiceBus
         #Receive a msg, prep it, and handle any errors that may occur
         # - Most of this should be queue independant
         def StartListeningToEndpoints
-            log "Waiting for messages. To exit press CTRL+C"
+            log 'Waiting for messages. To exit press CTRL+C'
             #            statOutputCountdown = 0
             messageLoop = true
             retries = @config.maxRetries
@@ -260,28 +260,28 @@ module RServiceBus
                     begin
                         @stats.incTotalProcessed
                         @msg = YAML::load(body)
-                        if @msg.msg.class.name == "RServiceBus::Message_Subscription" then
+                        if @msg.msg.class.name == 'RServiceBus::Message_Subscription' then
                             @subscriptionManager.add( @msg.msg.eventName, @msg.returnAddress )
-                            elsif @msg.msg.class.name == "RServiceBus::Message_StatisticOutputOn" then
+                            elsif @msg.msg.class.name == 'RServiceBus::Message_StatisticOutputOn' then
                             @stats.output = true
-                            log "Turn on Stats logging"
-                            elsif @msg.msg.class.name == "RServiceBus::Message_StatisticOutputOff" then
+                            log 'Turn on Stats logging'
+                            elsif @msg.msg.class.name == 'RServiceBus::Message_StatisticOutputOff' then
                             @stats.output = false
-                            log "Turn off Stats logging"
-                            elsif @msg.msg.class.name == "RServiceBus::Message_VerboseOutputOn" then
-                            ENV["VERBOSE"] = "true"
-                            log "Turn on Verbose logging"
-                            elsif @msg.msg.class.name == "RServiceBus::Message_VerboseOutputOff" then
-                            ENV.delete( "VERBOSE" )
-                            log "Turn off Verbose logging"
+                            log 'Turn off Stats logging'
+                            elsif @msg.msg.class.name == 'RServiceBus::Message_VerboseOutputOn' then
+                            ENV['VERBOSE'] = 'true'
+                            log 'Turn on Verbose logging'
+                            elsif @msg.msg.class.name == 'RServiceBus::Message_VerboseOutputOff' then
+                            ENV.delete('VERBOSE')
+                            log 'Turn off Verbose logging'
 
 
                             else
 
                             self.HandleMessage()
 
-                            if !@config.forwardReceivedMessagesTo.nil? then
-                                self._SendAlreadyWrappedAndSerialised(body,@config.forwardReceivedMessagesTo)
+                            unless @config.forwardReceivedMessagesTo.nil? then
+                              self._SendAlreadyWrappedAndSerialised(body, @config.forwardReceivedMessagesTo)
                             end
                         end
                         @mq.ack
@@ -307,16 +307,16 @@ module RServiceBus
                         #This has been re-rasied from a rescue in the handler
                         puts "*** #{e.message}"
                         #"Property, #{e.message}, not set for, #{handler.class.name}"
-                        propertyName = e.message[10, e.message.index(",", 10)-10]
+                        propertyName = e.message[10, e.message.index(',', 10)-10]
                         puts "*** Ensure the environment variable, RSB_#{propertyName}, has been set at startup."
 
                         rescue Exception => e
                         sleep 0.5
 
-                        puts "*** Exception occurred"
+                        puts '*** Exception occurred'
                         puts e.message
                         puts e.backtrace
-                        puts "***"
+                        puts '***'
 
                         if retries > 0 then
                             retries = retries - 1
@@ -326,19 +326,19 @@ module RServiceBus
                             @circuitBreaker.Failure
 
                             @stats.incTotalErrored
-                            if e.class.name == "Beanstalk::NotConnected" then
-                                puts "Lost connection to beanstalkd."
-                                puts "*** Start or Restart beanstalkd and try again."
+                            if e.class.name == 'Beanstalk::NotConnected' then
+                                puts 'Lost connection to beanstalkd.'
+                                puts '*** Start or Restart beanstalkd and try again.'
                                 abort();
                             end
 
-                            if e.class.name == "Redis::CannotConnectError" then
-                                puts "Lost connection to redis."
-                                puts "*** Start or Restart redis and try again."
+                            if e.class.name == 'Redis::CannotConnectError' then
+                                puts 'Lost connection to redis.'
+                                puts '*** Start or Restart redis and try again.'
                                 abort();
                             end
 
-                            errorString = e.message + ". " + e.backtrace.join( ". " )
+                            errorString = e.message + '. ' + e.backtrace.join('. ')
                             #                            log errorString
 
                             @msg.addErrorMsg( @mq.localQueueName, errorString )
@@ -349,7 +349,7 @@ module RServiceBus
                         end
                     end
                     rescue SystemExit, Interrupt
-                    puts "Exiting on request ..."
+                    puts 'Exiting on request ...'
                     messageLoop = false
 
                     rescue NoMsgToProcess => e
@@ -374,13 +374,13 @@ module RServiceBus
                     @circuitBreaker.Success
 
                     rescue Exception => e
-                    if e.message == "SIGTERM" || e.message == "SIGINT" then
-                        puts "Exiting on request ..."
+                    if e.message == 'SIGTERM' || e.message == 'SIGINT' then
+                        puts 'Exiting on request ...'
                         messageLoop = false
                         else
-                        puts "*** This is really unexpected."
+                        puts '*** This is really unexpected.'
                         messageLoop = false
-                        puts "Message: " + e.message
+                        puts 'Message: ' + e.message
                         puts e.backtrace
                     end
                 end
@@ -395,7 +395,7 @@ module RServiceBus
                 handlerList = @handlerManager.getHandlerListForMsg(msgName)
 
 
-                RServiceBus.rlog "Handler found for: " + msgName
+                RServiceBus.rlog 'Handler found for: ' + msgName
                 begin
                     @queueForMsgsToBeSentOnComplete = Array.new
 
@@ -409,7 +409,7 @@ module RServiceBus
                             raise PropertyNotSet.new( "Property, #{e.message}, not set for, #{handler.class.name}" )
                             rescue Exception => e
                             puts "E #{e.message}"
-                            log "An error occurred in Handler: " + handler.class.name
+                            log 'An error occurred in Handler: ' + handler.class.name
                             raise e
                         end
                     end
@@ -443,10 +443,10 @@ module RServiceBus
             # @param [String] serialized_object serialized RServiceBus::Message
             # @param [String] queueName endpoint to which the msg will be sent
             def _SendAlreadyWrappedAndSerialised( serialized_object, queueName )
-                RServiceBus.rlog "Bus._SendAlreadyWrappedAndSerialised"
+                RServiceBus.rlog 'Bus._SendAlreadyWrappedAndSerialised'
 
-                if !@config.forwardSentMessagesTo.nil? then
-                    @mq.send( @config.forwardSentMessagesTo, serialized_object )
+                unless @config.forwardSentMessagesTo.nil? then
+                  @mq.send(@config.forwardSentMessagesTo, serialized_object)
                 end
 
                 @mq.send( queueName, serialized_object )
@@ -457,14 +457,14 @@ module RServiceBus
             # @param [RServiceBus::Message] msg msg to be sent
             # @param [String] queueName endpoint to which the msg will be sent
             def _SendNeedsWrapping( msg, queueName, correlationId )
-                RServiceBus.rlog "Bus._SendNeedsWrapping"
+                RServiceBus.rlog 'Bus._SendNeedsWrapping'
 
                 rMsg = RServiceBus::Message.new( msg, @mq.localQueueName, correlationId )
-                if queueName.index( "@" ).nil? then
+                if queueName.index('@').nil? then
                     q = queueName
                     RServiceBus.rlog "Sending, #{msg.class.name} to, queueName"
                     else
-                    parts = queueName.split( "@" )
+                    parts = queueName.split('@')
                     rMsg.setRemoteQueueName( parts[0] )
                     rMsg.setRemoteHostName( parts[1] )
                     q = 'transport-out'
@@ -478,7 +478,7 @@ module RServiceBus
             def sendQueuedMsgs
                 @queueForMsgsToBeSentOnComplete.each do |row|
                     if row['timestamp'].nil? then
-                        self._SendNeedsWrapping( row["msg"], row["queueName"], row["correlationId"] )
+                        self._SendNeedsWrapping( row['msg'], row['queueName'], row['correlationId'] )
                     else
                         @sendAtManager.Add( row )
                     end
@@ -488,7 +488,7 @@ module RServiceBus
             def queueMsgForSendOnComplete( msg, queueName, timestamp=nil )
                 correlationId = @sagaData.nil? ? nil : @sagaData.correlationId
                 correlationId = (!@msg.nil? && !@msg.correlationId.nil?) ? @msg.correlationId : correlationId
-                @queueForMsgsToBeSentOnComplete << Hash["msg", msg, "queueName", queueName, "correlationId", correlationId, "timestamp",timestamp ]
+                @queueForMsgsToBeSentOnComplete << Hash['msg', msg, 'queueName', queueName, 'correlationId', correlationId, 'timestamp',timestamp ]
             end
 
             #Sends a msg back across the bus
@@ -497,7 +497,7 @@ module RServiceBus
             #
             # @param [RServiceBus::Message] msg msg to be sent
             def Reply( msg )
-                RServiceBus.rlog "Reply with: " + msg.class.name + " To: " + @msg.returnAddress
+                RServiceBus.rlog 'Reply with: ' + msg.class.name + ' To: ' + @msg.returnAddress
                 @stats.incTotalReply
 
                 self.queueMsgForSendOnComplete( msg, @msg.returnAddress )
@@ -509,9 +509,9 @@ module RServiceBus
 
                 return @mq.localQueueName if @handlerManager.canMsgBeHandledLocally(msgName)
 
-                log "No end point mapping found for: " + msgName
-                log "**** Check environment variable MessageEndpointMappings contains an entry named : " + msgName
-                raise "No end point mapping found for: " + msgName
+                log 'No end point mapping found for: ' + msgName
+                log '**** Check environment variable MessageEndpointMappings contains an entry named : ' + msgName
+                raise 'No end point mapping found for: ' + msgName
             end
 
 
@@ -520,7 +520,7 @@ module RServiceBus
             #
             # @param [RServiceBus::Message] msg msg to be sent
             def Send( msg, timestamp=nil )
-                RServiceBus.rlog "Bus.Send"
+                RServiceBus.rlog 'Bus.Send'
                 @stats.incTotalSent
 
                 msgName = msg.class.name
@@ -533,7 +533,7 @@ module RServiceBus
             #
             # @param [RServiceBus::Message] msg msg to be sent
             def Publish( msg )
-                RServiceBus.rlog "Bus.Publish"
+                RServiceBus.rlog 'Bus.Publish'
                 @stats.incTotalPublished
 
                 subscriptions = @subscriptionManager.get(msg.class.name)
@@ -547,7 +547,7 @@ module RServiceBus
             #
             # @param [String] eventName event to be subscribes to
             def Subscribe( eventName )
-                RServiceBus.rlog "Bus.Subscribe: " + eventName
+                RServiceBus.rlog 'Bus.Subscribe: ' + eventName
 
                 queueName = self.getEndpointForMsg( eventName )
                 subscription = Message_Subscription.new( eventName )