Seguridad en encriptado de datos en la DDBB

https://theobjective.com/espana/2024-04-15/ciberataque-expuestos-datos-personales-guardia-civil/

Un ciberataque deja expuestos los datos personales de toda la Guardia Civil y el Ejército

El ataque cibernético ha afectado a una empresa de reconocimientos médicos subcontratada por Interior y Defensa

y vosotros preocupados porque se filtren algunos hashes de números de teléfono aleatorios de usuarios

ke os flipais mucho los frikis ai con vuestros hash pero luego os meto un navajazo y no hay encriptado que os salve hahhahah

#43 no es para autenticar, si no para crear una nueva cuenta. se quiere encontrar un metodo que impida a un trol crearse 50 cuentas

#44 Si el teléfono solo lo necesitas para el registro no necesitas almacenar ninguna información que relaciones el usuario con el teléfono.

Que el código recibido en el tfno se compruebe con el esperado y si coincide se da el OK al registro y se almacena exclusivamente el número de teléfono para no poder volver a recibir otro registro.

AikonCWD pide registro con el numero 666111222, codigo de registro C46Fv&gds%/d. AikonCWD mete el codigo al registrarse y tirando.
Mediavida almacena 666111222 como numero utilizado
AikonCWD es baneado
AikonCWD2.0 pide registro con el numero 666111222. Mediavida devuelve que el numero de tfno no es válido, pero sin saber quien lo utilizó en su momento.

¿Esto no es viable?

#46 creo que eso sería valido

Una PoC muy simple:

import hashlib
import os


hashes_and_salts = []

def hash_private_key(private_key):
    salt = os.urandom(16)
    hash_obj = hashlib.pbkdf2_hmac('sha256', private_key.encode(), salt, 100000)
    return hash_obj, salt

def process_private_key(private_key):
    new_hash, new_salt = hash_private_key(private_key)
    collision_detected = False
    for old_salt, old_hash in hashes_and_salts:
        regenerated_hash = hashlib.pbkdf2_hmac('sha256', private_key.encode(), old_salt, 100000)
        if regenerated_hash == old_hash:
            collision_detected = True
            print("Collision detected with previously used private key.")
            break

if not collision_detected:
    hashes_and_salts.append((new_salt, new_hash))
    print("No collision detected: New hash and salt stored.")

print(f"Generated Hash: {new_hash.hex()}, Salt: {new_salt.hex()}")

process_private_key('phone1') 
process_private_key('phone1') 
process_private_key('phone2') 

Output:

Generated Hash: f3c2793b7aef9e94fcba7b32ccccf4d21087837e47d1e3a8820c39076e4e6fcf, Salt: 5bde7dd3d6bc4f4836e2d20106958276
No collision detected: New hash and salt stored.
Generated Hash: 91d6c9dc7ae73ca5740758e0f591079618a465634ea45e726473d98bb6b70761, Salt: f28076a8c7bca40ab2a82da4f76f42ef
Collision detected with previously used private key.
Generated Hash: 91746c89c505f4839797559a6a1824ff00f8b97d61852fa196c7414fab193679, Salt: df11d03a66bce4ddf21aa75831143b9c
No collision detected: New hash and salt stored.

edit: habria que hacer las private_key inteligentes, no solo un número, y usar argon2 claro. aqui es donde me suena que el standard es usar un doble salt. fijaros que podriamos encriptar nuestro hash del servicio de la misma manera con otro salt, y de esta manera poder cambiar los hashes si nunca se leakea o tener rotaciones de seguridad.

#45 Los números de teléfono si se reutilizan, no se lo que tardan en volverlos a dar una vez los dan de baja, pero usarse se utilizan xD

Por cierto, deberían de estar asociados al usuario, ya que si decide darse de baja voluntariamente, su información (entre la que se incluye el teléfono) debería de borrarse tambien.

#50 Si no he entendido mal tu código, para comprobar si ya se ha utilizado antes, generas un HASH pasando por todos los SALTs que ya tienes almacenados.

Es decir, que si hubieras guardado previamente 30mil teléfonos, para agregar uno nuevo tendrías que generar 30mil hash para comprobar que no se ha guardado previamente, lo cual, dependiendo del tiempo de generación de cada uno de los hash, sería inviable.

#53 En el caso de que te quiten hasta el código, daría un poco igual, pero si es solo la db, no valoras lo del "blind index"?

No manejo mucho de python, pero según internet quedaría un código más o menos así con los comentarios que he hecho anteriormente (el tema del coste es para que la generación tarde más y poder evitar los casos de fuerza bruta, algo así como bcrypt pero a lo cutre)

import base64
import hashlib

def generateBlindIndex(string, cost):
    value = string

for i in range(cost * 100000):
    parts = [value[j:j+3] for j in range(0, len(value), 3)]
    encodedParts = [base64.b64encode(part.encode()).decode() for part in parts]
    key = ''.join(encodedParts)
    value = hashlib.sha256(key.encode()).hexdigest()

return value

# Ejemplo de uso
string = "example_string"
cost = 1
blind_index = generateBlindIndex(string, cost)
print(blind_index)

PD: En mi local funciona (mi local es una página warra de internet que compila python online xD)
PD2: el "code" de mediavida no indenta donde debe:

El debate que se esta formando aqui y luego en el mundo real he visto hasta guardado de contrase;as en texto plano en la base de datos, incluida la direccion y el telefono xD