irufus / gdax-java

Java based wrapper for Coinbase Pro (Formerly known as GDAX API and Coinbase Exchange API)

MIT License

177 stars 131 forks source link

fix issue #55 missing User-Agent header in API calls and issue #58 getFill API change #57

Closed onyxcoyote closed 5 years ago

onyxcoyote commented 5 years ago

Fixes issue where cloudflare blocks API requests with no User-Agent header. Based on the comment by alistairrutherford, the value of the User-Agent does not seem to matter, as long as it's present.

"Access denied | api-public.sandbox.pro.coinbase.com used Cloudflare to restrict access"

onyxcoyote commented 5 years ago

Added a fix for issue #58. All the tests should be working with this pull.

58: OrderService.getAllFills() method was removed because that request is no longer supported by the coinbase pro API. It was replaced with getFillsByProductId() and getFillByOrderId()

robevansuk commented 5 years ago

Merged