braintree / braintree_java

Braintree Java library
https://developer.paypal.com/braintree/docs/start/overview
MIT License
158 stars 98 forks source link

Adding http logging capabilities #37

Closed paveliva closed 8 years ago

paveliva commented 8 years ago

Hi,

During our project we have faced with an issue that there is no chance to log request/response info. It is necessary for us to prevent and investigate possible issues in the future. And also we think that it can be quite useful for the library.

Please approve our merge request.

Thanks

pblesi commented 8 years ago

Hi @paveliva Thank you for your code submission. We think these changes are valuable and worth pursuing.

We ask that you make a few changes to your implementation

Thanks!

omgrr commented 8 years ago

Looks like these changes are being addressed in a newly opened pull request. I'm going to close this one and track future efforts on that PR. Be sure to comment there if you have any concerns.