axman6 / amazonka-s3-streaming

Provides a conduit based interface to uploading data to S3 using the Multipart API
MIT License
20 stars 23 forks source link

Replace $ with % in printf formatting strings. #7

Closed JakeOShannessy closed 7 years ago

JakeOShannessy commented 7 years ago

The format strings used for logging contain $ instead of % for some variables. This causes a runtime error when logging is enabled.

axman6 commented 7 years ago

Good catch, I'll fix that now!

axman6 commented 7 years ago

So I completely missed that this was a PR and just fixed it, with a version bump and updated the changelog. Not sure how I managed to not have this crash on me, guess I didn't test logging after rewriting for 0.2.