search

nicolindemann / coinbase-moneymoney

Fetches balances from Coinbase API and returns them as securities
MIT License
19 stars 10 forks source link

Error on Account refresh #8

Open martink2 opened 3 years ago

martink2 commented 3 years ago

Hi,

since a couple of days i get an error refreshing account data, logs show the following:

          1> Sending: GET https://api.coinbase.com/v2/exchange-rates?currency=REPV2
10:56:38  1> HTTPS response: Bad Request
          1> The server responded with the error message: Bad Request. Retrying...
          1> Sending: GET https://api.coinbase.com/v2/exchange-rates?currency=REPV2
          1> HTTPS response: Bad Request
          1> The API server of your bank responded with the error message: »Bad Request«  Please try again later or contact the support of your bank with this error message.

all previous requests for other currencies seem to go through ok:

1> Sending: GET https://api.coinbase.com/v2/exchange-rates?currency=COMP
1> Received: {"data":{"currency":"COMP","rates":{"AED":"424.585188","AFN":"8906.21042472","ALGO":"337.046216649657418254","ALL":"12004.45276629","AMD":"55670.01979452","ANG":"208.1463807","AOA":"75690.29703","AR

Let me know if i can provide more Info, not sure how to increase debug level.

slauber commented 3 years ago

There seems to be no data for REPv2 currency exchange information available via the Coinbase API.

One approach to solve this issue would be to exclude this specific currency from this plugin, a more stable approach would be to catch the exception, display no exchange rate and carry on.

iSchorsch commented 3 years ago

While the fetching worked like a charm between December 2020 and 15th of January 2021, it is unfortunately broken again. Is it like that just on my side?

HTTPS response: Bad Request The server responded with the error message: Bad Request. Retrying... Sending: GET https://api.coinbase.com/v2/exchange-rates?currency=XRP HTTPS response: Bad Request Der API-Server Ihrer Bank antwortet mit der Fehlermeldung: »Bad Request« Bitte versuchen Sie es später noch einmal oder wenden Sie sich mit dieser Fehlermeldung an den Support Ihrer Bank.

marothe commented 3 years ago

I still have the same problem. Was the PR ever approved and merged?

slauber commented 3 years ago

Hi @marothe, this repo was forked and includes my PR as well as a fix for the issue @iSchorsch raised https://github.com/mnin/coinbase-moneymoney