support multiple To's in sendMail

[extrawurst]

No description provided.

[s-ludwig]

Thanks. I think the only thing missing now for fully correct support is to do foreach (addr; v.splitter(",").map(addr => addr.strip())), because a comma separated list is what RFC 5322 specifies. I couldn't find a mention of multiple "To" headers, but I guess that any client will merge those to a comma separated list according to RFC 822, so it should be equivalent.

Is that something you'd also like to add, or should I quickly do that?

[extrawurst]

@s-ludwig no I am fine, go for it if you like ;)

[extrawurst]

ok what the heck i gonna do it, I need this fix ;) we are just talking about the stuff thats written inside the DATA segment, right ? so after writing the TOs I will simply do you convert code ?!

[s-ludwig]

I'll merge and make the change. My first message already took longer to type ;)
There will be no heap/GC allocations, so anything in the DATA segment will be used as is.

[extrawurst]

IMHO this syntax rules here: https://www.ietf.org/rfc/rfc0822.txt
seem to suggest that indeed multiple "To" in the DATA segment are valid anyway

[s-ludwig]

Ah OK now I get it. I thought you meant something with the DATA segment of the executable.

The only question now is if RFC 5322 still supports multiple To headers, or if that's deprecated. At least it always talks about those fields in singular and section 4.5 seems to imply that multiple occurrences might be deprecated now, but I didn't find a concrete statement.