Skip to content

Commit 53be04a

Browse files
authored
Merge pull request #75 from veverkap/master
This adds in logging for parameters
2 parents 87acf01 + f9b4cfe commit 53be04a

File tree

2 files changed

+13
-0
lines changed

2 files changed

+13
-0
lines changed

lib/tesla/middleware/logger.ex

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ defmodule Tesla.Middleware.DebugLogger do
2828
env
2929
|> log_request
3030
|> log_headers("> ")
31+
|> log_params("> ")
3132
|> log_body("> ")
3233
|> Tesla.run(next)
3334
|> log_response
@@ -53,6 +54,13 @@ defmodule Tesla.Middleware.DebugLogger do
5354
env
5455
end
5556

57+
def log_params(env, prefix) do
58+
for {k,v} <- env.query do
59+
_ = Logger.debug "#{prefix} Query Param '#{k}': '#{v}'"
60+
end
61+
env
62+
end
63+
5664
def log_body(%Tesla.Env{} = env, _prefix) do
5765
Map.update!(env, :body, & log_body(&1, "> "))
5866
end

test/tesla/middleware/logger_test.exs

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,4 +42,9 @@ defmodule LoggerTest do
4242
log = capture_log(fn -> Client.get("/ok") end)
4343
assert log =~ "/ok -> 200"
4444
end
45+
46+
test "ok with params" do
47+
log = capture_log(fn -> Client.get("/ok", query: %{"test" => "true"}) end)
48+
assert log =~ "Query Param 'test': 'true'"
49+
end
4550
end

0 commit comments

Comments
 (0)