Add logger to api requests

Victor Costa · Rodrigo Souto
1 parent df13e6de
Showing 9 changed files with 17 additions and 10 deletions Show diff stats
lib/api/api.rb
lib/api/helpers.rb
lib/api/v1/articles.rb
lib/api/v1/categories.rb
lib/api/v1/comments.rb
lib/api/v1/communities.rb
lib/api/v1/enterprises.rb
lib/api/v1/people.rb
lib/api/v1/users.rb
@@ -3,6 +3,11 @@ Dir[&quot;#{Rails.root}/lib/api/*.rb&quot;].each {|file| require file}
 module API
   class API < Grape::API
+    before { start_log }
+    before { setup_multitenancy }
+    before { detect_stuff_by_domain }
+    after { end_log }
+
     version 'v1'
     prefix "api"
     format :json
@@ -3,7 +3,7 @@ module API
     PRIVATE_TOKEN_PARAM = :private_token
     def logger
-      API.logger
+      @logger ||= Logger.new(File.join(Rails.root, 'log', "#{ENV['RAILS_ENV']}_api.log"))
     end
     def current_user
@@ -168,6 +168,17 @@ module API
     end
     protected
+    def start_log
+      logger.info "Started #{request.path}"
+    end
+    def end_log
+      logger.info "Completed #{request.path}"
+    end
+
+    def setup_multitenancy
+      Noosfero::MultiTenancy.setup!(request.host)
+    end
+
     def detect_stuff_by_domain
       @domain = Domain.find_by_name(request.host)
       if @domain.nil?
 module API
   module V1
     class Articles < Grape::API
-      before { detect_stuff_by_domain }
       before { authenticate! }
       resource :articles do
 module API
   module V1
     class Categories < Grape::API
-      before { detect_stuff_by_domain }
       before { authenticate! }
       resource :categories do
 module API
   module V1
     class Comments < Grape::API
-
-      before { detect_stuff_by_domain }
       before { authenticate! }
       resource :articles do
 module API
   module V1
     class Communities < Grape::API
-      before { detect_stuff_by_domain }
       before { authenticate! }
       resource :communities do
 module API
   module V1
     class Enterprises < Grape::API
-      before { detect_stuff_by_domain }
       before { authenticate! }
       resource :enterprises do
 module API
   module V1
     class People < Grape::API
-      before { detect_stuff_by_domain }
       before { authenticate! }
       resource :people do
 module API
   module V1
     class Users < Grape::API
-   
-      before { detect_stuff_by_domain }
       before { authenticate! }
       resource :users do
1	module API	1	module API
2	module V1	2	module V1
3	class Articles < Grape::API	3	class Articles < Grape::API
4	- before { detect_stuff_by_domain }
5	before { authenticate! }	4	before { authenticate! }
6		5
7	resource :articles do	6	resource :articles do