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

Merged
zelvuska merged 1 commits from malo-mailu-z-testwebu into master 1 year ago
ledoian commented 1 year ago
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 1 year ago
cfdbcc8ee1 Omezení na počet adres v hlavičce mailů z testwebu
Owner

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

Intuitivně bych čekal, že ta funkce bude vracet list, ale to asi vidím na první pohled, že to bude string.
Poster
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ě…
zelvuska merged commit 1521b9d192 into master 1 year ago
zelvuska deleted branch malo-mailu-z-testwebu 1 year ago
The pull request has been merged as 1521b9d192.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.