Merge pull request #4109
2951436 wallet: warn when payment IDs are used (moneromooo-monero)
This commit is contained in:
@@ -1557,6 +1557,7 @@ void wallet2::process_new_transaction(const crypto::hash &txid, const cryptonote
|
||||
{
|
||||
// We got a payment ID to go with this tx
|
||||
LOG_PRINT_L2("Found encrypted payment ID: " << payment_id8);
|
||||
MINFO("Consider using subaddresses instead of encrypted payment IDs");
|
||||
if (tx_pub_key != null_pkey)
|
||||
{
|
||||
if (!m_account.get_device().decrypt_payment_id(payment_id8, tx_pub_key, m_account.get_keys().m_view_secret_key))
|
||||
@@ -1580,12 +1581,9 @@ void wallet2::process_new_transaction(const crypto::hash &txid, const cryptonote
|
||||
else if (get_payment_id_from_tx_extra_nonce(extra_nonce.nonce, payment_id))
|
||||
{
|
||||
LOG_PRINT_L2("Found unencrypted payment ID: " << payment_id);
|
||||
MWARNING("Found unencrypted payment ID: these are bad for privacy, consider using subaddresses instead");
|
||||
}
|
||||
}
|
||||
else if (get_payment_id_from_tx_extra_nonce(extra_nonce.nonce, payment_id))
|
||||
{
|
||||
LOG_PRINT_L2("Found unencrypted payment ID: " << payment_id);
|
||||
}
|
||||
|
||||
for (const auto& i : tx_money_got_in_outs)
|
||||
{
|
||||
|
||||
Reference in New Issue
Block a user