darkarp / chromepass

Chromepass - Hacking Chrome Saved Passwords
GNU Affero General Public License v3.0
777 stars 109 forks source link

Code enhancement #31

Closed arrzdev closed 3 years ago

arrzdev commented 3 years ago

Sugestão "steal.py" Linhas "138-142"

STEAL.PY

LINES 138-142

Código demonstrativo:

#dummy var
name = "t-name"
decrypted = "t-decrypted"
url = "t-url"

#init dict
cookies = {"t-url":[]}

##############################################

#before
cookie_obj = {}
cookie_obj["name"] = name
cookie_obj["value"] = decrypted
cookie_obj["domain"] = url

cookies[url].append(cookie_obj)

#before output
print(f"BEFORE: {cookies}")

##############################################

#reset dict
cookies[url] = []

#after
cookies[url].append(
    {
        "name": name,
        "value": decrypted,
        "domain": url
    }
)

#after output
print(f"AFTER: {cookies}")

Não é nada muito significativo e de todo muda a forma como o código se comporta, mas assim fica escrito de forma mais "pythonizada" e obtemos um codigo com uma leitura simplificada

@arrz#8714

darkarp commented 3 years ago

Mudei para ser um pouco mais "pythonic". Contudo, nao gosto de colocar objectos construidos dentro de um append, prefiro separa o objecto e o append, fica menos messy.