Add logger to api requests

Victor Costa · Rodrigo Souto
1 parent 80dec54f
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
...	...	@@ -3,6 +3,11 @@ Dir["#{Rails.root}/lib/api/*.rb"].each {\|file\| require file}
3	3
4	4	module API
5	5	class API < Grape::API
	6	+ before { start_log }
	7	+ before { setup_multitenancy }
	8	+ before { detect_stuff_by_domain }
	9	+ after { end_log }
	10	+
6	11	version 'v1'
7	12	prefix "api"
8	13	format :json
...	...
...	...	@@ -3,7 +3,7 @@ module API
3	3	PRIVATE_TOKEN_PARAM = :private_token
4	4
5	5	def logger
6		- API.logger
	6	+ @logger \|\|= Logger.new(File.join(Rails.root, 'log', "#{ENV['RAILS_ENV']}_api.log"))
7	7	end
8	8
9	9	def current_user
...	...	@@ -168,6 +168,17 @@ module API
168	168	end
169	169	protected
170	170
	171	+ def start_log
	172	+ logger.info "Started #{request.path}"
	173	+ end
	174	+ def end_log
	175	+ logger.info "Completed #{request.path}"
	176	+ end
	177	+
	178	+ def setup_multitenancy
	179	+ Noosfero::MultiTenancy.setup!(request.host)
	180	+ end
	181	+
171	182	def detect_stuff_by_domain
172	183	@domain = Domain.find_by_name(request.host)
173	184	if @domain.nil?
...	...
1	1	module API
2	2	module V1
3	3	class Articles < Grape::API
4		- before { detect_stuff_by_domain }
5	4	before { authenticate! }
6	5
7	6	resource :articles do
...	...
1	1	module API
2	2	module V1
3	3	class Categories < Grape::API
4		- before { detect_stuff_by_domain }
5	4	before { authenticate! }
6	5
7	6	resource :categories do
...	...
1	1	module API
2	2	module V1
3	3	class Comments < Grape::API
4		-
5		- before { detect_stuff_by_domain }
6	4	before { authenticate! }
7	5
8	6	resource :articles do
...	...
1	1	module API
2	2	module V1
3	3	class Communities < Grape::API
4		- before { detect_stuff_by_domain }
5	4	before { authenticate! }
6	5
7	6	resource :communities do
...	...
1	1	module API
2	2	module V1
3	3	class Enterprises < Grape::API
4		- before { detect_stuff_by_domain }
5	4	before { authenticate! }
6	5
7	6	resource :enterprises do
...	...
1	1	module API
2	2	module V1
3	3	class People < Grape::API
4		- before { detect_stuff_by_domain }
5	4	before { authenticate! }
6	5
7	6	resource :people do
...	...
1	1	module API
2	2	module V1
3	3	class Users < Grape::API
4		-
5		- before { detect_stuff_by_domain }
6	4	before { authenticate! }
7	5
8	6	resource :users do
...	...