-
Notifications
You must be signed in to change notification settings - Fork 235
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
Invalid nonce when reusing keys between applications #75
Comments
Does the same happen if you load the key from a file (using Are you sure the key/secret are valid? Did you previously use them, especially with a different library? |
I tested using the load_keys() method and I received the same error. I also tested the keys introducing them into my coinigy.com account and they worked just fine. The public queries also work properly. The problem is with the private calls. |
Did you use the same key/secret pair when trying both Coinigy and The former may be using a different Have you tried a wholly new key/secret? |
Yes, I used the same key/secret pair in Coinigy and Krakenex. They work with Coinigy, but not with krakenex. As I am trying to use krakenex for a Hedge Fund account, setting new API keys is a complex process as we use dozens and dozens of API keys. It´s simpler for me to learn how to do all the authentication process by myself and writting my own wrapper than asking for new API keys. Definitely it's a weird behavior, especially if krakenex works with other API keys and Coinigy works with my API keys. If I make it work, I'll notify you. |
Multiplying time.time() by 1000000 instead of 1000 fixed the problem for my key/secret pair. |
I had the same problem described by @pauljherrera and followed his suggested approach. After doing this, I'm able to retrieve balance info. Kraken recently refactored their site and it's possible they are requiring a larger nonce # for new API users. I had never used my API for any private requests until this. Thanks @pauljherrera ! |
I tried the same suggestion, but it does not make a difference for me. Same error: |
@jsgv I failed to include one important piece of information in my prior comment. I updated the krakenex I did this because the Kraken servers use UTC time and I believe the nonce errors may be the result of a millisecond value being sent which is lower than the server millisecond nonce. Using UTC along with the 1000000 works consistently now. Try these updates and let us know if it solves your problem. |
@FergusClare is this still working for you? I'm still getting the error with your recommended change. Edit: I created a new key pair and everything works now. |
The
This seems to confirm the earlier guess of Coinigy using a different numbering scheme. Reusing API keys between services is not a good idea in general - precisely for the reason that you're going to get bugs like this one; not getting deeper on security implications. Therefre closing as "wontfix". If you'd like to discuss changing the |
Actually - sorry! - reopening with an edited, more verbose title; so that there's a clear indication of an unsupported use pattern. The issue would likely be brought up again otherwise. |
I had the same problem. |
@pietrosperoni That seems to me like an overly-complicated way of dealing with unsynced time, in a scenario where keys are still being reused. (I find this inappropriate from a security perspective also.) What can be done is |
Versions
What are you trying to achieve?
Retrieving account balance.
What do you expect to happen?
Get a valid response.
What happens instead?
Nonce error.
The text was updated successfully, but these errors were encountered: