coinbase/coinbase-ios-sdk

Is there a bug with Requesting Money ??

tr33z0r opened this issue · 25 comments

tr33z0r commented

Is there anyone having issues with requesting money?

https://api.coinbase.com/v2/accounts/:account_id/transactions

I've debugged using advanced rest client and I'm still getting this error:

{"errors":[{"id":"param_required","message":"Missing parameter: type"}]}

Here is the json I'm sending across (email is redacted):

{
"type": "request",
"to": “email@email.com”,
"amount": "1",
"currency": “USD”
}

bchavez commented

Hello Coinbase,

I currently maintain the C# and .NET Coinbase API library available here: https://github.com/bchavez/Coinbase

I can also confirm this is an issue with the Coinbase REST API.

The following image below shows the HTTP network traffic from the Fiddler debugging tool on Windows. Fiddler displays the results of attempting to make a "Request Money" operation on Coinbase according to the REST documentation available here: https://developers.coinbase.com/api/v2#request-money

Fiddler_3548

As shown above, all the required parameters are specified as the API documentation describes. However, an HTTP 400 is returned by Coinbase.

The error is described as:

{"errors":[{"id":"param_required","message":"Missing parameter: `type`"}]}

Which doesn't make sense because the type parameter does exist in the HTTP request body.

Also, I've tried various CB-VERSION dates, and the response is still the same.

The unit test case that generated the request is as follows:

[Test]
public async Task can_send_mony_request()
{
   var accounts = await client.Accounts.ListAccountsAsync();

   var btcAccount = accounts.Data.First(a => a.Name.Contains("BTC Wallet"));

   var requestMoney = new RequestMoney
      {
         Type = "request",
         Amount = 1,
         Currency = "BTC",
         Description = "Test request",
         To = "bchavez@gmail.com"
      };

   var moneyRequest = await client.Transactions.RequestMoneyAsync(btcAccount.Id, requestMoney);

   moneyRequest.Dump();
}

Thanks,
Brian

Reopened related issue posted July 25th, 2019: bchavez/Coinbase#57

  • 👍2
punidramesh commented

Is anyone looking into this?

  • 👍2
Mike-E-angelo commented

Is anyone looking into this?

But more importantly, does anyone at Coinbase care anymore? Why create an API that has very obviously broken functionality and not bother to fix it for many years now? And here I thought PayPal was bad. 😅😭

That $100B valuation has people already checking out... if they weren't already checked out before, that is. 🤷‍♂️

Mike-E-angelo commented

Personally, I just now received a reply from Coinbase "support" for a ticket that was sent April 25th. Only 39 days, not bad. Just kidding that is terrible. :P

https://howlongagogo.com/date/2021/april/25

Not only did they not answer the question, they spelled my name wrong. 🤦‍♂️

pmatar28code commented

Hello Coinbase,

I currently maintain the C# and .NET Coinbase API library available here: https://github.com/bchavez/Coinbase

I can also confirm this is an issue with the Coinbase REST API.

The following image below shows the HTTP network traffic from the Fiddler debugging tool on Windows. Fiddler displays the results of attempting to make a "Request Money" operation on Coinbase according to the REST documentation available here: https://developers.coinbase.com/api/v2#request-money

Fiddler_3548

As shown above, all the required parameters are specified as the API documentation describes. However, an HTTP 400 is returned by Coinbase.

The error is described as:

{"errors":[{"id":"param_required","message":"Missing parameter: `type`"}]}

Which doesn't make sense because the type parameter does exist in the HTTP request body.

Also, I've tried various CB-VERSION dates, and the response is still the same.

The unit test case that generated the request is as follows:

[Test]
public async Task can_send_mony_request()
{
   var accounts = await client.Accounts.ListAccountsAsync();

   var btcAccount = accounts.Data.First(a => a.Name.Contains("BTC Wallet"));

   var requestMoney = new RequestMoney
      {
         Type = "request",
         Amount = 1,
         Currency = "BTC",
         Description = "Test request",
         To = "bchavez@gmail.com"
      };

   var moneyRequest = await client.Transactions.RequestMoneyAsync(btcAccount.Id, requestMoney);

   moneyRequest.Dump();
}

Thanks,
Brian

Reopened related issue posted July 25th, 2019: bchavez/Coinbase#57

Hello, did you finally get a solution to this? , im facing the same problem, thanks.

Mike-E-angelo commented

I received an email that Coinbase Commerce will now be charging a 1% fee for using their broken API. 🤷‍♂️

Mike-E-angelo commented

LULZ... my comments were minimized. Nice to see someone over there is actually alive and "cares" :P

ispysoftware commented

I think it's probably a github bot you should be thanking.
thinking about using coinbase here and a lot concerned that this issue from freaking 2019 hasn't even been responded to

  • 👍1
Mike-E-angelo commented

Yep, I do feel bad for being such a drag, but there is a good reason why I am doing so: I am building an application on this "platform" and it has holes in it. So, not much of a platform at all, or a stable one at that.

In fact, I am looking at using BitPay instead now because of this issue. 😭

jwitcig commented

Hey all,
I am really sorry about this having gone unaddressed. Unfortunately, we have made the decision to sunset this SDK, and will no longer be providing support going forward.


If this is still a concern, we recommend checking out the refreshed documentation for Coinbase Cloud. If the new docs leave you with further questions, the developer forum there is a great place to start a discussion. You can also share feedback in the forum about anything that you think we might need to look into.

Best of luck on your endeavors!