Add logger to api requests

Victor Costa
1 parent d9c13a65
Showing 9 changed files with 17 additions and 11 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
@@ -166,6 +166,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
@@ -19,7 +18,6 @@ module API
 #      :params => API::Entities::Article.documentation
 #    }
         get do
-  
           articles = select_filtered_collection_of(environment, 'articles', params)
           present articles, :with => Entities::Article 
         end
 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
...	...	@@ -166,6 +166,17 @@ module API
166	166	end
167	167	protected
168	168
	169	+ def start_log
	170	+ logger.info "Started #{request.path}"
	171	+ end
	172	+ def end_log
	173	+ logger.info "Completed #{request.path}"
	174	+ end
	175	+
	176	+ def setup_multitenancy
	177	+ Noosfero::MultiTenancy.setup!(request.host)
	178	+ end
	179	+
169	180	def detect_stuff_by_domain
170	181	@domain = Domain.find_by_name(request.host)
171	182	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
...	...	@@ -19,7 +18,6 @@ module API
19	18	# :params => API::Entities::Article.documentation
20	19	# }
21	20	get do
22		-
23	21	articles = select_filtered_collection_of(environment, 'articles', params)
24	22	present articles, :with => Entities::Article
25	23	end
...	...
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
...	...