-
Notifications
You must be signed in to change notification settings - Fork 48
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
bug: uploading a directory fails due to excessive payments being deducted which repeatedly drain the wallet #1957
Comments
Hi @happybeing , Thank you for reporting this and providing all the context and files on the Autonomi forum! I have just tried uploading the
Could you please confirm that your issue was fixed on the latest client? |
Thanks @mickvandijke I'll need a solution to this before I can test any fix against my own client: https://forum.autonomi.community/t/how-to-build-a-client-app-against-the-correct-version-of-the-network/40089?u=happybeing Can you help with that? |
EDIT: and I'll also need some nanos, BTW: did your test use a wallet with just enough tokens according to the estimated cost? The problem doesn't arise if you have a wallet with a whole token (eg from a faucet), because the excessive cost doesn't cause the upload to fail. So any test should happen with a wallet that has enough nanos but not a billion nanos. With a few nanos (say a couple of thousand I think) in the wallet, each time it failed it still charged the wallet for chunks that were not uploaded. Also be aware that if those chunks are already on the network the test won't be valid. Can you confirm that this issue was found and fixed explicitly? Or are you just testing with the latest client and thinking, ah the problem isn't happening any more? |
Absolutely agree with you that the instructions with the keys and version number are a bit unclear. Could you perhaps open an issue with the same contents you posted here: https://forum.autonomi.community/t/how-to-build-a-client-app-against-the-correct-version-of-the-network/40089?u=happybeing ? I'll bring it up with the team. |
I did test it with the latest client and 2 full tokens initially. I have also tried it again with a new wallet that only had 850 nanos and new files in a folder in a similar structure
If you still think I'm testing it wrong, let me know! Also, if you want to test again for yourself, but need some nanos, please send me your address here or on Discord (my username is @mick_maidsafe) and I'll send you some. |
Sounds like a good test 😁 phew! So fingers crossed. I will now need to sort the client build issue. One thing:
I noticed the above seems a bit odd. Ignoring that I opened the issue as requested, which you've probably seen. Once I am able to build my client to connect to the current beta I will need some nanos. I may be able to get enough from my own nodes which someone is kindly running on my behalf, but if I do need more I will be in touch. So thanks for that! |
Also, if you want to test again for yourself, but need some nanos, please send me your address here or on Discord (my username is @mick_maidsafe) and I'll send you some. Actually @mickvandijke please can you send me say 3000 nanos so I'm ready to start testing as soon as I get the client connecting again:
I already have a donation of 311 and expect more from the community and my nodes, but having a few ready in the wallet would help. Also, please can you send a friend request to me on Discord as happybeingagain. I'm reluctant to try friending anyone there now as it caused a lot of hassle when I tried friending Bux and my account was suspended. She has now friended me instead. Thank you 🙏 |
Thanks very much for your help. Things are improved and I've been able to upload chunks with plenty of nanos to spare, although so far only 20 out of the 76 chunks for my new test because of other problems. The original bug appears to be fixed though, so I will close this. 💪 I've opened a new issue for what I'm seeing now which is that it is stopping partway through with a panic that appears to be in the crates used by autonomi. For more see: #1979 |
This occurs with
safe files upload -p <DIRECTORY>
as well as with a client app I'm developing which uses the FilesApi.Report here, including uploaded logs: https://forum.autonomi.community/t/assemble-at-the-start-line-the-beta-is-about-to-begin/39882/568?u=happybeing
The text was updated successfully, but these errors were encountered: