Change puts to logger and define global var logger from BigBlueButtonApi

Author: JoaoCarAlmSilva

lib/bigbluebutton_api.rb

@@ -9,6 +9,7 @@
 require 'bigbluebutton_modules'
 require 'bigbluebutton_config_xml'
 require 'bigbluebutton_config_layout'
+require 'logger'
 
 module BigBlueButton
 
@@ -53,6 +54,9 @@ class BigBlueButtonApi
     # Flag to turn on/off debug mode
     attr_accessor :debug
 
+    # logger to log reponses and infos
+    attr_accessor :logger
+
     # Maximum wait time for HTTP requests (secs)
     attr_accessor :timeout
 
@@ -75,13 +79,18 @@ def initialize(url, secret, version=nil, debug=false)
       @timeout = 10         # default timeout for api requests
       @request_headers = {} # http headers sent in all requests
 
+      @logger = Logger.new(STDOUT)
+      @logger.level = Logger::DEBUG
+
       version = nil if version && version.strip.empty?
       @version = nearest_version(version || get_api_version)
       unless @supported_versions.include?(@version)
-        puts "BigBlueButtonAPI: detected unsupported version, using the closest one that is supported (#{@version})"
+        @logger.warn("BigBlueButtonAPI: detected unsupported version, using the closest one that is supported (#{@version})")
+        # puts "BigBlueButtonAPI: detected unsupported version, using the closest one that is supported (#{@version})"
       end
 
-      puts "BigBlueButtonAPI: Using version #{@version}" if @debug
+      @logger.debug("BigBlueButtonAPI: Using version #{@version}")
+      # puts "BigBlueButtonAPI: Using version #{@version}" if @debug
     end
 
     # Creates a new meeting. Returns the hash with the response or throws BigBlueButtonException
@@ -743,7 +752,8 @@ def send_api_request(method, params={}, data=nil, raw=false)
     # Otherwise uses GET
     def send_request(url, data=nil)
       begin
-        puts "BigBlueButtonAPI: URL request = #{url}" if @debug
+        @logger.debug("BigBlueButtonAPI: URL request = #{url}")
+        # puts "BigBlueButtonAPI: URL request = #{url}" if @debug
         url_parsed = URI.parse(url)
         http = Net::HTTP.new(url_parsed.host, url_parsed.port)
         http.open_timeout = @timeout
@@ -753,11 +763,14 @@ def send_request(url, data=nil)
         if data.nil?
           response = http.get(url_parsed.request_uri, @request_headers)
         else
-          puts "BigBlueButtonAPI: Sending as a POST request with data.size = #{data.size}" if @debug
+          @logger.debug("BigBlueButtonAPI: Sending as a POST request with data.size = #{data.size}")
+          # puts "BigBlueButtonAPI: Sending as a POST request with data.size = #{data.size}" if @debug
           opts = { 'Content-Type' => 'application/xml' }.merge @request_headers
           response = http.post(url_parsed.request_uri, data, opts)
         end
-        puts "BigBlueButtonAPI: URL response = #{response.body}" if @debug
+        # @logger.debug("BigBlueButtonAPI: URL response = #{response.body}")
+        @logger.info("BigBlueButtonAPI: request=#{url} response_status=#{response.class.name} response_code=#{response.code}  message_key=#{response.message}")
+        # puts "BigBlueButtonAPI: URL response = #{response.body}" if @debug
 
       rescue Timeout::Error => error
         raise BigBlueButtonException.new("Timeout error. Your server is probably down: \"#{@url}\". Error: #{error}")