-
selsta
.merge+ 8296
-
xmr-pr
Added
-
MeowingCat
what does crypto_core_ed25519_scalar_reduce() do?
-
moneromooo
Probably reduce a scalar, which means modulo... some large number. Canonicalization.
-
MeowingCat
loooks like MyMonero is hashing private spend key with Keccak256 for private view key
-
rbrunner
MyMonero in particular? Isn't this just the normal, general way how view private keys are derived from spend private keys in all of Monero?
-
endogenic
yes, I have a PR for it
-
endogenic
16 byte seed support
-
endogenic
old PR
-
endogenic
i will reintroduce it though
-
endogenic
i upgraded mymonero away from those seeds a few yrs ago
-
endogenic
but clients still need support
-
MeowingCat
Base58("18" + PUB_SPEND_KEY + PUB_VIEW_KEY + Keccak256("18" + PUB_SPEND_KEY + PUB_VIEW_KEY)[0:8])
-
MeowingCat
doing this
-
MeowingCat
butttt
-
MeowingCat
reduce function is coming from libsodium thing
-
MeowingCat
-
MeowingCat
this thing
-
MeowingCat
looks simple
-
MeowingCat
for porting lol
-
rbrunner
What are you trying to port, and from where to where?
-
MeowingCat
implementing HD wallet thing for Monero in C#
-
MeowingCat
im gonna port libsodium's sc25519_reduce()
-
MeowingCat
i think building TX structures and signing will be very difficult
-
rbrunner
I am a bit confused. If you implement a general "HD wallet thing", why do you check what MyMonero is doing? Why not check what the Monero core software does?
-
MeowingCat
im looking everywhere but we will use lesser words like MyMonero
-
rbrunner
Use lesser words?
-
MeowingCat
oh i mean short private seed
-
moneromooo
If you want to use yet another seed format, you'll just confuse and annoy people.
-
moneromooo
And we're apparently also going to :D
-
MeowingCat
yess but words are also used for BTC and ETH
-
MeowingCat
i can use more words and split for others if needed
-
MeowingCat
i think these things are not in big endian because of networking or something
-
MeowingCat
damn C# can't just easily interpete byte array parts as something like int
-
MeowingCat
millions of things
-
MeowingCat
these things are making me crazy
-
MeowingCat
i wanna sleep
-
rbrunner
By all means, if you want to implement seeds with less words, implement this:
github.com/tevador/polyseed
-
rbrunner
Anything else is just looking for disaster and confusion, IMHO
-
mj-xmr[m]
jeffro256: please join Community
-
mj-xmr[m]
We're discussing your CCS proposal
-
selsta
monero-project/monero #8360 can someone check if this makes sense? should if there is an rpc error, should there also be an error with the HTTP status code instead of 200?
-
selsta
that was badly formulated.. looking into the github issue should make it clear
-
selsta
-
MeowingCat
-
MeowingCat
how can i get public spend key here?
-
MeowingCat
i need to compare my derivation
-
MeowingCat
for same seed
-
selsta
-
MeowingCat
thank youuuuuuuuuuuuuuuu
-
MeowingCat
_crypto_scalarmult_ed25519_base()
-
MeowingCat
it is different than mine
-
MeowingCat
damn
-
MeowingCat
isn't private spend key scalar reduced derivation of private seed?
-
MeowingCat
MyMonero is not doing that
-
MeowingCat
it is doing scalar reduce to Keccak256 hash of private seed
-
endogenic
old mymonero, you mean?
-
endogenic
what are you referencing
-
endogenic
i mentioned historically mymonero had 13 word seeds i migrated them away from
-
endogenic
mymonero-core-cpp account.cpp shows how you do it in c++
-
endogenic
i mean the account c++ it uses
-
MeowingCat
as i see current MyMonero
-
MeowingCat
im gonna do zero padding to private seed
-
MeowingCat
i think it must be little endian
-
endogenic
how are you "seeing" current mymonero. it's compiled wasm
-
MeowingCat
oh Github source
-
endogenic
link?