Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Does not sync anymore #7

Closed
iSchorsch opened this issue Nov 24, 2020 · 12 comments · May be fixed by #9
Closed

Does not sync anymore #7

iSchorsch opened this issue Nov 24, 2020 · 12 comments · May be fixed by #9

Comments

@iSchorsch
Copy link

For more than a week, the plug in does not sync anymore. Checked the API settings on the webSite of CoinBase. Everything seems to be correct - used to work before.

Error message:

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.

@ncserny
Copy link

ncserny commented Nov 24, 2020

Same issue here. Please check.
Screen Shot 2020-11-24 at 11 05 15

@smnkrl
Copy link

smnkrl commented Nov 24, 2020

Same issue here - it always fails at the same stage:

1> Sending: GET https://api.coinbase.com/v2/exchange-rates?currency=REPV2
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> 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.

@jvgh1
Copy link

jvgh1 commented Nov 26, 2020

Same here:

Sending: GET https://api.coinbase.com/v2/exchange-rates?currency=REPV2; HTTPS response: Bad Request

@udochrist
Copy link

i have changed the code to remove the request for REPv2 as this is a not allowed contract to watch for me.
however my knowledge of lua coding as well as coinbase api is not to far adanced.
Besides i do not know how to generate the correct signature so this ext can be used w/o turning of sigs in the beta version....
if you want i can generate a PR

@iSchorsch
Copy link
Author

Thank you very much for your effort! Did you try your modification and does it work? If it fetches your account correctly I would be very happy if you could share your extension, even if we would have to enable ß-support / non signed extensions.
I am curious if the original developer of this great Lua is still able to respond?

@udochrist
Copy link

Thank you very much for your effort! Did you try your modification and does it work? If it fetches your account correctly I would be very happy if you could share your extension, even if we would have to enable ß-support / non signed extensions.
I am curious if the original developer of this great Lua is still able to respond?

This is my quick fix. It simply skips over REPV2.

https://github.com/udochrist/coinbase-moneymoney/blob/fixREPv2/src/Coinbase.lua

@iSchorsch
Copy link
Author

Great! This does work now. Thank you! thumbs up

@fabge
Copy link

fabge commented Nov 28, 2020

How were you able to add a extension without the right signature?
Were can I turn the checking for signature off?

@mr089
Copy link

mr089 commented Nov 28, 2020

This is my quick fix. It simply skips over REPV2.

https://github.com/udochrist/coinbase-moneymoney/blob/fixREPv2/src/Coinbase.lua

it works again. many thanks

@udochrist
Copy link

udochrist commented Nov 28, 2020 via email

@smnkrl
Copy link

smnkrl commented Nov 30, 2020

MoneyMoney provided a new signed version of the extension - it works fine for me.
https://moneymoney-app.com/extensions/

20201130162339

@iSchorsch
Copy link
Author

super! Bei mir auch! DANKE!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants