stripe / stripe-ruby / 2923

94%

master: 97%

push

travis-ci

Remove attempt to nicen `UploadIO` in logs

In #741 I tried to do something too clever by replacing instances of
`Faraday::UploadIO` found in parameters with a human-readable string to
improve `STRIPE_LOG` logging output.

I thought I'd tested it at the time, but apparently not (or not well
enough), and this change caused the regression detailed in #742.

My findings about how Faraday encodes multipart were apparently wrong
and it does use these parameters, so here we remove the step where we
try to nicen them for logging. The logs look a little worse, but it's
not that big of a deal.

I've tested this patch against the API and confirmed that it addresses
the problem.

Fixes #742.

1 of 1 new or added line in 1 file covered. (100.0%)

1636 of 1734 relevant lines covered (94.35%)

3226.54 hits per line