-
tusko
ELI5 how checkpointing is necessary and not a privacy concern?
-
jeffro256[m]
Old block probably wont change. Make hash of hash so sync go fast. Not privacy concern b/c why would it be
-
tusko
I'm sure IP addresses would at least be exchanged between nodes and the DNS operators, would they not?
-
jeffro256[m]
Only if you use DNS checkpoints
-
tusko
So the tradeoff is use DNS checkpoints and always stay on the 'right' chain but lose some privacy, or maintain privacy but security model is fucked without DNS checkpoints?
-
tusko
How is MoneroPulse not a trusted third-party?
-
jeffro256[m]
<tusko> "So the tradeoff is use DNS..." <- The daemon also has hardcoded checkpoints built right into the binary. If you don't use DNS checkpoints, your sync will be a little slower, that's it
-
selsta
the last checkpoint on moneropulse seems at height 1680000
-
selsta
that was years ago, it's basically unused
-
jeffro256[m]
Yeah honestly that feature can probably be removed IMO but the node will sync just fine with checkpoints
-
jeffro256[m]
*without
-
tusko
Yeah, I guess my question then is when will it be deprecated? AFAICT nodes still phone home so-to-speak
-
gingeropolous
i think it can be useful in case of emergencies
-
gingeropolous
not just for speeding up.
-
tusko
If it should remain for some incomprehensible reason, then why can't the callback to the centralized servers at least be sent over Tor.
-
gingeropolous
dunno. make it happen
-
tusko
I might do
-
sech1
.merges
-
xmr-pr
8794 8805 8808 8810 8811 8813
-
sech1
Can everyone take a look at these PRs one final time and maybe it's time to finally release?
-
jeffro256[m]
8794 needs to be opened against release
-
jeffro256[m]
moneromooo:
-
moneromooo
selsta: do you want that for 0.18 ?
-
moneromooo
jeffro256[m]: I just saw your last comment, updated.
-
selsta
moneromooo: luigi somehow only merged the one against release
-
selsta
-
selsta
so we need the last change against release
-
moneromooo
Oh it already open against 0.18. OK.
-
moneromooo
Last change you mea just the assert then, right ?
-
moneromooo
I'll undo that assert change since I did not realize it had been merged to 0.18 yet.
-
moneromooo
Less confusing to have matching patches.
-
jeffro256[m]
Oh I didn’t see release got merge either oops
-
jeffro256[m]
Yeah Probably better to keep them matching
-
jeffro256[m]
I’ve got a PR ooen which addresses that assert condition but it’s not as important as the rest of the PR
-
jeffro256[m]
I don't know anything about Windows systems programming, so I can't vouch for/against 8810 & 8811, but everything else looks good
-
sech1
I didn't know about this specific _beginthread() restriction either, I was just investigating why monerod ran out of resources on my PC
-
selsta
.merges
-
xmr-pr
8794 8805 8808 8810 8811 8813
-
selsta
moneromooo: there's a typo in 8794
-
selsta
`THROW_WALLET_EXCEPTION_IF` has two commas and build fails
-
moneromooo
fixed