TPCF-27209 - CAPI logs should for request complete should include the response time

Add time taken to json based on code review comments

Author: nookala

lib/cloud_controller/logs/request_logs.rb

@@ -29,10 +29,11 @@ def start_request(request_id, env)
       def complete_request(request_id, status, env, time_taken)
         return if @incomplete_requests.delete(request_id).nil?
         request = ActionDispatch::Request.new(env)
-        @logger.info("Completed #{status} vcap-request-id: #{request_id} Response Time: #{time_taken}", 
+        @logger.info("Completed #{status} vcap-request-id: #{request_id}", 
                      { status_code: status,
-                      request_method: request.request_method,
-                      request_fullpath: request.filtered_path })
+                       time_taken: time_taken,
+                       request_method: request.request_method,
+                       request_fullpath: request.filtered_path })
       end
 
       def log_incomplete_requests

spec/unit/lib/cloud_controller/logs/request_logs_spec.rb

@@ -47,7 +47,7 @@ module VCAP::CloudController::Logs
 
           it 'logs the completion of the request' do
             request_logs.complete_request(request_id, status, env, time_taken)
-            expect(logger).to have_received(:info).with(/Completed 200 vcap-request-id: ID Response Time: 30/, { request_method: "request_method", request_fullpath: "filtered_path", status_code: 200 })
+            expect(logger).to have_received(:info).with(/Completed 200 vcap-request-id: ID/, { time_taken: 30, request_method: "request_method", request_fullpath: "filtered_path", status_code: 200 })
           end
         end