Merge pull request #515 from Mashape/fix/mashape-analytics

[fix/alf-serializer] invalid 'url' property in ALF
Kong · Aug 27, 2015 · 61d4227 · 61d4227
2 parents 7f7f82d + 311388d
commit 61d4227
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/kong/plugins/log_serializers/alf.lua b/kong/plugins/log_serializers/alf.lua
@@ -120,7 +120,7 @@ function _M.serialize_entry(ngx)
     time = alf_time,
     request = {
       method = ngx.req.get_method(),
-      url = ngx.var.scheme.."://"..ngx.var.host..ngx.var.uri,
+      url = ngx.var.scheme.."://"..ngx.var.host..ngx.var.request_uri,
       httpVersion = "HTTP/"..ngx.req.http_version(),
       queryString = dic_to_array(ngx.req.get_uri_args()),
       headers = alf_req_headers_arr,

diff --git a/spec/plugins/mashape-analytics/fixtures/requests.lua b/spec/plugins/mashape-analytics/fixtures/requests.lua
@@ -17,7 +17,7 @@ return {
       var = {
         scheme = "http",
         host = "mockbin.com",
-        uri = "/request",
+        request_uri = "/request",
         request_length = 123,
         body_bytes_sent = 934,
         remote_addr = "127.0.0.1",
@@ -109,7 +109,7 @@ return {
       var = {
         scheme = "http",
         host = "mockbin.com",
-        uri = "/request",
+        request_uri = "/request",
         request_length = 123,
         body_bytes_sent = 934,
         remote_addr = "127.0.0.1",