Missing “SessionCreated” event in the event of an error at the secure endpoint

[tinohager]

If a secure session is established on port 587 but no data is transferred and the session times out, no event is triggered. Would there be an argument against triggering the event before the UpgradeAsync?

SmtpServer/Src/SmtpServer/SmtpSessionManager.cs

Lines 38 to 42 in 8a5bce4

	await UpgradeAsync(handle, linkedTokenSource.Token);
	linkedTokenSource.Token.ThrowIfCancellationRequested();
	_smtpServer.OnSessionCreated(new SessionEventArgs(handle.SessionContext));

[cosullivan]

Yes it probably makes sense to move it.

[tinohager]

I have made the changes