RFC: Omezení počtu adres v mailech z testwebu #27
Merged
zelvuska
merged 1 commits from malo-mailu-z-testwebu
into master
2 years ago
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'malo-mailu-z-testwebu'
Deleting a branch is permanent. It CANNOT be undone. Continue?
Není dobré poslat moc mailů v záhlaví mailu z testwebu. Typicky to není potřeba a je to potenciální riziko pro ty adresy, kdyby se k nim někdo někdy dostal.
Nemám to otestované, nemám ani nápad, jak to testovat (resp. tu funkci otestovat asi můžu), asi je nejjednodušší to vyzkoušet přímo na testwebu.
Ten kód není hezký, funkce se jmenuje divně. Je půlnoc, moje kreativita přestává být měřitelná :-)
Pokud někoho napadne nějaké vylepšení, tak ho klidně použiju :-)
Intuitivně bych čekal, že ta funkce bude vracet list, ale to asi vidím na první pohled, že to bude string.
Jenže ono IMHO moc nejde vrátit pole, které by se dalo nějak snadno rovnou použít. Buď vrátíš něco jako
maily[:2]
, resp.maily[:2] + ([...] if len(maily) > 3 else [])
a počet mailů si dopočítáš v původní funkci, a nebo z toho bude padat divné heterogenní pole které na nějaké pozici bude mít int, a pak bude středně pochybný kód na to to rozparsovat do výsledného stringu. Ani jedno za to IMHO spíš nestojí, když takhle je to relativně pohromadě…1521b9d192
into master 2 years ago1521b9d192
.