Weird naming `NewSMTPClient`
IceflowRE opened this issue · 3 comments
IceflowRE commented
NewSMTPClient
should be NewSMTPServer
or at least it should be documented, why this is how it is.
Foosec commented
If i could add, its not just the use of the "server" variable,
a function called NewSMTPClient returns a SMTPServer
Perhaps the function should be changed to NewSMTPServer, although i understand the predicament with that aswell.
Maybe NewSMTPEndpoint?
Just my 2c