-
MeowingCat
does wallet2 API have something to save current state of synchronized chain?
-
MeowingCat
Monero::WalletListener::newBlock() is giving current height
-
MeowingCat
but i need to save current state
-
TrasherDK[m]
monerod release on mainnet ignoring stop_daemon command.
-
TrasherDK[m]
screen -S "${SNAME}-${NETTYPE}" -X stuff "stop_daemon\n"
-
TrasherDK[m]
7fc2ce7fc700 INFO msgwriter src/common/scoped_message_writer.h:102 Stop signal sent
-
TrasherDK[m]
A kill -TERM $(pidof seceensession) had to be used, to force a shutdown. Attaching the session issuing "stop_daemon", "exit" or CTRL-C had no effect.
-
TrasherDK[m]
This is what a scripted shutdown should look like:
-
TrasherDK[m]
7f89e4aea700 INFO msgwriter src/common/scoped_message_writer.h:102 Stop signal sent... (full message at
libera.ems.host/_matrix/media/r0/do…6e5bca1dd117dcb11b2a3a69cbce8b9ad81)
-
MeowingCat
hiiiiiiiiii
-
MeowingCat
virtual bool store(const std::string &path) = 0;
-
MeowingCat
i think thissssssssss
-
moneromooo
If you can repro it, a thread dump would be useful (gdb /path/to/monerod ${pid-of-the-waiting-monerod} then: thread apply all bt).
-
moneromooo
I also think the net patch by wfaressuissia on github fixes it.
-
moneromooo
(one such bug anyway)
-
TrasherDK[m]
It's currently working as expected. It's not something I can provoke. The daemon is launched every 30 minutes, and shut down when synced. I will try if it hangs again.
-
MeowingCat
Monero::Wallet::store() doesn't store blocks
-
MeowingCat
isn't there anything to save blocks in wallet2???????????
-
moneromooo
It saves block hashes. Is that enough ?
-
MeowingCat
it is enough to synchronize right??
-
moneromooo
Yes.
-
MeowingCat
but how does it save hashes??
-
moneromooo
By... writing to disk...
-
MeowingCat
ohhhh
-
MeowingCat
i mean what should i do to make it save?
-
moneromooo
Call store, which you found. It's done automatically when you exit the wallet normally.
-
MeowingCat
just tried
-
moneromooo
^C is deemed normal for the purposes of this.
-
MeowingCat
but it doesn't save blocks
-
MeowingCat
when i restart the app
-
MeowingCat
ohhhhh
-
MeowingCat
waitt
-
MeowingCat
wallet->setRefreshFromBlockHeight()
-
MeowingCat
maybe about this?
-
MeowingCat
im doing store() buttt it doesn't save or still getting blocks from node
-
moneromooo
As I said, it never saves blocks. Only block hashes. If you want it to save blocks for some reason, you'll have to add that code. And your wallet won't be compatible with monero-wallet-cli.
-
MeowingCat
i mean saving "current state"
-
MeowingCat
im doing wallet->store("monero-wallet");
-
MeowingCat
it saves monero-wallet.keys
-
moneromooo
Hmm. Maybe I'm confusing things. Lemme check the code...
-
MeowingCat
i mean should this save current state and supposed to not retrieve blocks from node when i restart the app???
-
moneromooo
OK, if can write the keys file if you store as another name, but if not, it'll save the cache file only.
-
moneromooo
store will save the current state to the wallet cache file, and is not supposed to retrieve blocks fom the node.
-
moneromooo
When you restart the wallet program, it will typically try to sync to the node.
-
moneromooo
That depends on the program. And if you've disabled auto refresh.
-
moneromooo
But usually, it'll try to.
-
MeowingCat
it is doing refresh
-
MeowingCat
"refreshed" callback is being called
-
MeowingCat
you mean doing wallet->store() is enough to save current state?
-
moneromooo
It should be.
-
MeowingCat
really confusing
-
moneromooo
What do you think might be a less confusing API to save it ?
-
MeowingCat
im doing wallet->store() but when i restart the app, it is getting blocks from node
-
moneromooo
You can disable auto refresh if you want.
-
moneromooo
Or do you mean it's downloading *all* blocks from the genesis block again ?
-
MeowingCat
no not from genesis block the block that i specify with wallet->setRefreshFromBlockHeight()
-
moneromooo
Is your wallet setup to trust the daemon ?
-
MeowingCat
you mean if i disable auto refresh it will store current state and start from current state when i restart?
-
moneromooo
If your daemon is on 127.0.0.1, it'll be a yes.
-
MeowingCat
wm->setDaemonAddress("
xmr.moneromerch.com:18089");
-
MeowingCat
im doing this
-
MeowingCat
wallet->init("
xmr.moneromerch.com:18089", 0, "", "", true, false, "");
-
MeowingCat
and this
-
moneromooo
That call doesn't quite match the prototype I have here, but I'll assume the daemon is untrusted since it's not on localhost.
-
moneromooo
In that case, the wallet will quantize its current height and start asking the daemon from blocks a bit earlier than needed, for privacy reasons.
-
moneromooo
It's to remove some bits of fingerprinting ability. Though there's probably a lot more left.
-
moneromooo
IIRC the quantum is about 1000 blocks. So when you've synced at most ~1k more blocks, it'll start asking from blocks from a later starting point.
-
moneromooo
Use your own node on 127.0.0.1 and it won't do tha.
-
MeowingCat
interestinggg
-
MeowingCat
the app has to use a remote node
-
MeowingCat
it will use our node
-
MeowingCat
and start synchronizing from wallet creation time
-
MeowingCat
is there a way to make wallet2 to trust node??
-
moneromooo
Then it'll get a bit more traffic at start. No biggie, the wallet won't actually rescan those txes IIRC.
-
moneromooo
Which in itself can be an info leak, since the faster it goes htrough txes, the faster it can ask for more, so the node can try to make a guess, but hey.
-
moneromooo
Yes. See wallet2.h.
-
moneromooo
It's also a param in init.
-
moneromooo
Here, it'll mean that the user of the wallet trusts you. Not that you trust yourself.
-
MeowingCat
virtual bool init(const std::string &daemon_address, uint64_t upper_transaction_size_limit = 0, const std::string &daemon_username = "", const std::string &daemon_password = "", bool use_ssl = false, bool lightWallet = false, const std::string &proxy_address = "") = 0;
-
MeowingCat
you mean this init()?
-
moneromooo
Yes. Thought it was a parameter here, I misremembered.
-
moneromooo
Hrm. My copy has it. Must be a recent change then.
-
moneromooo
Still in master. What class is that init function you're quoting in ?
-
moneromooo
Ah, found it. wallet2_api. Makes sense.
-
moneromooo
I was looking at wallet2.
-
MeowingCat_
meowwwwww
-
MeowingCat
damnnn
-
MeowingCat
i merged my C++ static library and all wallet2 libraries with ar now build time is better
-
MeowingCat
i wrote a Makefile
-
MeowingCat
i love GNU Makeeeee
-
MeowingCat
what could we do if MSYS2 was not exist
-
MeowingCat
anyone doesn't care about building Windows
-
MeowingCat
building on Windows*
-
MeowingCat
using Bash commands and Linux things
-
MeowingCat
Godot takes care about building on different platforms
-
moneromooo
You could... use Linux. In fact, one might even go as far as you could use Linux even if MSYS2 exists.
-
monerobull[m]
how big is the blockchain file right now?
-
fluffypony
at least 1
-
fluffypony
at [17:32:15] ➜ ls -lh ~/.bitmonero/lmdb/
-
fluffypony
total 276351632
-
fluffypony
-rw-r--r-- 1 ric waffles 132G Jun 11 03:59 data.mdb
-
monerobull[m]
thanks, im currently trying to get the blockchain data out of my docker node and into my pinode
-
moneromooo
Less than 50 GB if pruned.
-
fluffypony
can you prune -> move -> unprune?
-
moneromooo
You cannot unprune.
-
fluffypony
we could call it growing
-
fluffypony
:-P
-
moneromooo
fluffing
-
fluffypony
lol
-
fluffypony
I guess the real issue with a Pinode is HDD thrash when syncing / importing from .raw, so probably the best thing to do is just to move it whole
-
garth
Good point.
-
garth
It’s worth asking, at some point, if a Rasp pi has ever become fundamentally inadequate to be supported. Here’s an interesting thread:
monero-ecosystem/PiNode-XMR #65
-
garth
Well it’s more of a single post than a thread
-
ofrnxmr[m]
-
ofrnxmr[m]
Uh. Make that 1/2/3
-
MeowingCat
moneromooo, i was a Linux catttttt
-
MeowingCat
in past but now im a Windows cattt
-
MeowingCat
im using WSL instead of Linux
-
MeowingCat
im making TXs
-
MeowingCat
i should do Monero::PendingTransaction::commit() right???
-
MeowingCat
tx->txid() returns a std::vector but it is empty
-
MeowingCat
as i understand TXIDs are being cleared after tx->commit()
-
MeowingCat
but tx->txid() is still empty before tx->commit()
-
MeowingCat
but whyyyy
-
MeowingCat
ohh tx->status() is 1
-
MeowingCat
my TradeOgre address starts with 8
-
MeowingCat
shouldn't that start with 4?
-
moneromooo
Subaddresses all start with 8 (on mainnet). They must be using them.
-
MeowingCat
where is subaddress index placed in the private seed?
-
MeowingCat
or it is not placed in private seed
-
MeowingCat
i should loook at my key derivation implementation
-
MeowingCat
string pub_key = (int)Network + Convert.ToHexString(PublicSpendKey) + Convert.ToHexString(PublicViewKey);
-
moneromooo
The seed generates all subaddresses. That's a metric fuckton of them. You don't want indices for all of these in your seed.
-
MeowingCat
but where should i specify subaddress indexes?
-
moneromooo
In account_index when calling transfer, typically.
-
moneromooo
Wherever the API has a parameter for one, really.
-
MeowingCat
i mean i want to allow generating subaddresses tooo in my C# library
-
moneromooo
That API would take a subadress index. Pass it one.
-
moneromooo
Maybe I'm missing your point :)
-
MeowingCat
ummmmm
-
MeowingCat
i mean im deriving Monero keys includes public address tooo
-
MeowingCat
but i want to derive subaddresses tooo
-
MeowingCat
where is Monero's word list?
-
MeowingCat
found
-
wernervasquez[m]
-
wernervasquez[m]
-
MeowingCat
wernervasquez[m], tyyy
-
wernervasquez[m]
MeowingCat: I tried to keep my code lightweight and readable. The wallet.go file has some of the logic like scanning. Please let me know if anything is not clear enough so I can update it to be more clear.
-
ooo123ooo1234567
<sech1> "so probably some new or updated..." <- is it important ? there is a fix for it
-
selsta
sech1: will look into it
-
selsta
at least bisect the change that introduced this, might need help for the actual fix
-
» moneromooo unignores sech1, what's the bug ?
-
selsta
-
selsta
some change in master branch increased it
-
sech1
yes, latest master branch makes Ubuntu 16.04 sad
-
sech1
Official v0.17.3.2 works fine though
-
moneromooo
nm -a "build/Linux/cc/release/bin/townforge" | grep @@GLIBC_ | tr @ \ | sort -k3,3V | tail
-
moneromooo
That gives you the symbols that uses the most recent glibc.
-
moneromooo
I see getrandom being new...
-
selsta
getentropy, explicit_bzero, __explicit_bzero_chk, reallocarray
-
selsta
that's the one i'm seeing for 2.25 and 2.26
-
moneromooo
What's your target ? .23 ?
-
selsta
we previously had 2.17
-
Monegro
Is it normal to get a ton of repeated "connection refused" on the log when trying to connect to node.moneroworld.com
-
Monegro
?
-
Monegro
It's occassionally downloading some blocks
-
selsta
try a different node
-
Monegro
Any one in particular recommended for connecting over Tor?
-
w[m]
Ask in #monero:monero.social
-
w[m]
Ill answer you there
-
moneromooo
There are patches on github that improve connection stability over tor. Use those.
-
moneromooo
Look for PRs From jberman[m].
-
Monegro
thanks
-
selsta
moneromooo: these patches are for p2p tor, i think they were asking for rpc over tor
-
selsta
rpc over tor should be similar to normal rpc traffic
-
moneromooo
Ah, thanks.
-
selsta
but the patches are merged to master now so no one has to worry about it
-
selsta
.merge+ 7774
-
xmr-pr
Added
-
selsta
sech1:
monero-project/monero #7773 introduced the increased glibc dependency
-
selsta
not sure yet how to continue
-
sech1
Try older unbound versions?
-
selsta
i can try, but we updated it in the first place because unbound had an audit and was full of security issues
-
sech1
Drop support for Ubuntu 16? It's been more than a year since Ubuntu 16 end of life
-
selsta
will check tomorrow if going back with the unbound version helps, otherwise sometimes something like
monero-project/monero #6397 is also possible
-
moneromooo
You could check unbound config to see if there's a backward compat option. Not super likely though.
-
selsta
-
selsta
they have some compatibility code
-
selsta
but have to look tomorrow
-
ooo123ooo1234567
<selsta> ".merge+ 7774" <- it isn't clear from approval whether correctness of lmdb changes was verified or not.
-
ooo123ooo1234567
monero-project/monero #8381, the same here both approvals before important change and after aren't clear enough to understand whether correctness of patch overall was verified, though this change is certainly more isolated than lmdb related one