RFC: Omezení počtu adres v mailech z testwebu #27

Merged

zelvuska merged 1 commit from malo-mailu-z-testwebu into master

2023-04-17 20:55:11 +02:00

ledoian commented

2023-03-08 00:05:55 +01:00

Owner

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 :-)

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 :-)

ledoian added 1 commit 2023-03-08 00:05:56 +01:00

Omezení na počet adres v hlavičce mailů z testwebu cfdbcc8ee1

Není úplně žádoucí, aby se v mailu z testwebu poslala spousta adres,
protože je to náchylné na automatické zpracování v případě, že se k
takovému mailu dostane někdo další.

zelvuska commented

2023-03-13 21:12:54 +01:00

Owner

Intuitivně bych čekal, že ta funkce bude vracet list, ale to asi vidím na první pohled, že to bude string.

ledoian commented

2023-03-14 19:06:16 +01:00

Author

Owner

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ě…

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ě…