[ARVADOS] updated: 1.1.0-102-g105c22f

Git user git at public.curoverse.com
Mon Nov 6 15:14:30 EST 2017


Summary of changes:
 services/api/app/controllers/application_controller.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

       via  105c22f44dad79968b59f577a40737ffc8da00ec (commit)
      from  d6c06f757baee640f3cfd620bdb5a01d4ee8d4d2 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 105c22f44dad79968b59f577a40737ffc8da00ec
Author: Lucas Di Pentima <ldipentima at veritasgenetics.com>
Date:   Mon Nov 6 17:13:57 2017 -0300

    12533: Use @remote_ip to take into account the reverse proxy
    
    Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <ldipentima at veritasgenetics.com>

diff --git a/services/api/app/controllers/application_controller.rb b/services/api/app/controllers/application_controller.rb
index 3c786ea..9826cf2 100644
--- a/services/api/app/controllers/application_controller.rb
+++ b/services/api/app/controllers/application_controller.rb
@@ -388,7 +388,7 @@ class ApplicationController < ActionController::Base
   def append_info_to_payload(payload)
     super
     payload[:request_id] = response.headers['X-Request-Id']
-    payload[:client_ipaddr] = request.remote_ip
+    payload[:client_ipaddr] = @remote_ip
     payload[:client_auth] = current_api_client_authorization.andand.uuid || nil
   end
 

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list