-
sgp_
Is there any tool to take a Monero base address (4...) and add a 16 digit payment ID to it to make it an integrated address?
-
sgp_
I'm looking at a use-case where the SENDER would generate the pid and include it in the transaction
-
tobtoht
sgp_: here is some python code that'll do that:
paste.debian.net/plainh/46f1c459
-
sgp_
tobtoht: does what does `secrets.token_hex(8)`do specifically?
-
tobtoht
sgp_: generates a random 8 byte hex string like "6ce1e744b9e2bad8"
-
sgp_
what if I wanted to provide my string which was already generated?
-
tobtoht
PaymentID() takes a string up to a certain character limit
-
tobtoht
so you could do
-
tobtoht
my_pid = "payment_id_here"
-
tobtoht
PaymentID(my_pid)
-
sgp_
perfect
-
sgp_
how does the encryption of this short pid work?
-
sgp_
I assume it's decrypted with the private view key, but also I'm not 100% sure on that either
-
sgp_
also what does it mean in xmrchain when it says "Payment id (encrypted): e074da1078f11738"
xmrchain.net/tx/67831f68dc26c006353…3c82d1b59dd91fe9a67707968076cc2387d
-
sgp_
is e074da1078f11738 not the normal decrypted pid?
-
tobtoht
sgp_: if I am not mistaken the payment id embedded unencrypted in the address, but is encrypted with the public view key on the blockchain
-
sgp_
so if I was the recipient of 67831f68dc26c006353f5a8441f703c82d1b59dd91fe9a67707968076cc2387d, I would see a different pid other than e074da1078f11738? But which is the same length?
-
gingeropolous
so encrypted pids are staying?
-
tobtoht
sgp_: yes, payment ids in integrated addresses are always 64 bits / 8 bytes long
-
sgp_
gingeropolous: honestly that's a super good question, and if Thorchain can integrate using only encrypted pids, I think I'll change my mind and support its inclusion
-
sgp_
tobtoht: thank you, this is very helpful
-
gingeropolous
wtf is thorchain doing thats worth leaving screen doors on our submarine
-
sgp_
no KYC "DEX" swaps, synthetic XMR assets on other chains like ETH, etc
-
sgp_
personally I buy the use-case as "reasonable enough"
-
sgp_
others may disagree obviously
-
gingeropolous
i guess. seems to be all the rage. i still think being money is sorta the primary important thing ...
-
gingeropolous
can't they do what they need within the bp+ thinger?
-
sgp_
they most likely can when that's built out, sure. And when built, I would recommend switching to that (which they would probably be open to do)
-
sgp_
in <24 hours I've talked them down from adding 190 unique, unencrypted bytes in tx_extra to using the encrypted, standard-format pid. There's room to work within the lines of reason I think
-
gingeropolous
well ...... maybe we should just switch to bp+ before they go all out building stuff that we'll hope to deprecate within n years?
-
gingeropolous
i mean yeah sgp_ , thats awesome
-
sgp_
their timeline is <3 mo
-
sgp_
if BP+ encrypted messaging can be added before then, they will probably adopt it I think, if not, encrypted pid
-
gingeropolous
well i guess as long as they understand that at some point monero may force them to change their whole thing, then cool
-
sgp_
luckily it wouldn't be the "whole thing" for moving from a pid message to a BP+ message
-
sgp_
assuming that the BP+ message is decrypted by the private view key
-
gingeropolous
but man oh man, this kinda stuff is indicative that these holes have to be plugged
-
gingeropolous
also how many months is heart months? <3
-
sgp_
I completely agree that the totally wild west tx_extra should be addressed
-
sarang
BP can do data embedding too
-
gingeropolous
yeah, i thought it wasn't unique to BP+
-
sgp_
decrypted with the private view key, just to be 100% sure?
-
sgp_
^ for both encrypted pid and BP/BP+
-
tobtoht
sgp_: for payment ids see page 42 ZtoM2. The private view key and tx public key are required to decrypt the pid.
-
sarang
For data embedding you can use whatever construction you want
-
sarang
Presumably using the shared sender-recipient secret
-
sarang
It's a general construction
-
sarang
Note that it's 64 bytes per proof, not per recipient/output
-
sarang
And a transaction has a single proof
-
Inge-
sarang: I presume Triptych is not using unproven soundness assumptions? Arcturus is an excellent learning example of why making such assumptions is irresponsible when much is at stake.
-
moneromooo
64 bytes, not 32 ?
-
moneromooo
And I can add code to pass user data in BP+, that's something I wanted to look at at some point anyway.
-
moneromooo
This is also a good time to discuss "do we want a flat encrypted <insert number> byte data in every tx" again.
-
moneromooo
I have a patch for that on github, and it got kinda lost in discussion of the number (which was a function in my case).
-
sgp_
here is the spec that is being considered for Thorchain
gitlab.com/thorchain/thornode/-/issues/919
-
sgp_
they will take a sha256 hash of a command added to their blockchain and use the first 16 characters
-
sgp_
they want that to be passed along with a monero transaction
-
moneromooo
That'd fit snug in a BP+ extra data.
-
sgp_
sure, if the CLI wallet clearly showed that, then they would probably be super open to using that field instead
-
sgp_
does using the BP+ extra data increase the size of all transactions that use the field by 32 bytes? Or is this free space we aren't using right now
-
moneromooo
It doesn't exist yet, but I'm willing to add tooling for it.
-
moneromooo
It does not increase size.
-
moneromooo
We are using the space, but we can still fit something in.
-
» moneromooo tries to think of an analogy
-
sgp_
oh excellent, so we can pass this along for free. that's great news
-
sgp_
is it a huge amount of work to add the tooling?
-
moneromooo
Not huge. Not trivial. Medium maybe. For some handwavy definition of medium.
-
moneromooo
Low-medium.
-
moneromooo
At first approximation anyway.
-
moneromooo
Though if they're fine with 16 characters, assuming it's 16 hex chars, that one fits in the short payment id.
-
sgp_
they need 16 characters to match against the first 16 characters of a sha256 hash
-
moneromooo
That is however probably shit since I assume the full string will be public, so it becomes trivial to map it back.
-
sgp_
so my impression is that fits in an encrypted payment ID
-
moneromooo
However, if thay payment id is encrypted, then it's OK I think.
-
sgp_
they need a monero transaction to pass along a string that looks like this: 2aab7b9a80d9d65f
-
moneromooo
Depends whether only the recipient needs to decode it, or whether it needs to be publicly known.
-
moneromooo
The later case -> bad user
-
moneromooo
latter
-
sgp_
it doesn't need to be publicly known on the monero chain, but it will be made public since the receiving wallet view key is public
-
sgp_
sha256 hashes are all hex right?
-
moneromooo
They can be printed out as hex, like any other data.
-
moneromooo
I guess we need to discuss whether we want to prevent that before they start doing it.
-
moneromooo
Or maybe it's just some "use this at your own risk".
-
sgp_
how would you prevent it? even if you yank out tx_extra, they could use the BP message
-
moneromooo
Yes, but the harder it is to use, the less likely it us to be used :)
-
moneromooo
Would those public wallets' key images be publicly known ?
-
sgp_
if anything it's a reason to bump the ringsize I think in anticipation
-
sgp_
yup key images known too 😬
-
moneromooo
Fuckers...
-
sgp_
what needs to happen on a dev side before the next protocol update?
-
moneromooo
Merge BP+, I think it's not yet.
-
moneromooo
Possibly review/merge triptych if we want it in too.
-
moneromooo
But triptych would be best with shiny bins for rings, which we do not have yet.
-
moneromooo
And noone doing this AFAIK.
-
moneromooo
Oh, and no multisig yet anyway.
-
moneromooo
So no triptych.
-
moneromooo
ArticMine's fee changes.
-
sarang
Inge-: Triptych uses a weighted key aggregation, but not in the same way as Arcturus
-
sarang
That being said, it should still be noted that Triptych hasn't undergone a formal audit either
-
sarang
And this is a good opportunity to review the aggregation used in Triptych
-
Inge-
right. But Monero does reviews before implementing. Did Turtlecoin or someone put Arcturus into production?
-
sarang
AFAIK TurtleCoin had code and was planning a deployment
-
sarang
Arcturus preprint has been withdrawn:
eprint.iacr.org/2020/312
-
sgp_
So BP+, fee changes, BP+ 32 byte storage, ringsize bump?
-
gingeropolous
tx_extra phaseout?
-
gingeropolous
and this might be more for dev
-
sgp_
Sure let's move there
-
hyc
phaseout sounds untenable
-
hyc
since tx_extra is apparently essential to mining
-
Inge-
but it is encrypted, no? So it is not a privacy issue, more a tx size issue?
-
endor00[m]
<sarang "Arcturus preprint has been withd"> ErCiccione: maybe you should include a note about this on the MRL page
getmonero.org/resources/research-lab
-
ErCiccione
endor00: thanks for the ping. Will do.
-
sarang
Good call endor00[m], thanks for noting this
-
sarang
The existing IACR link will continue to work indefinitely (they keep old versions for a historical record)
-
sarang
but the "current version PDF" link will not exist (you need to view previous versions to get the PDF)
-
dEBRUYNE
sarang: As a side note, have you already started delving into the multisig research for Triptych?
-
sarang
Slowly but surely!
-
gingeropolous