threads / patch / 14386

patchgit-send-email: Fix authenticating on some servers when using TLS.

Subject: [PATCH] git-send-email: Fix authenticating on some servers when using TLS.

## tl;dr

2 messages between Jul 9, 2008 and Jul 9, 2008. Diffs are folded; open one to read it.

replies: 1people: 2as markdown or json

Robert Shearman· Jul 9, 2008, 21:39 UTC · lore
Send hello after a successful STARTTLS command to refresh the list of extensions. These may be different to what is returned over a clear connection (for example the AUTH command may be accepted over a secure connection, but not over a clear connection).
Furthermore, this behaviour is recommended by RFC 2487 (http://www.ietf.org/rfc/rfc2487.txt).
Signed-off-by: Robert Shearman <robertshearman@gmail.com>
---
 git-send-email.perl |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)
Show changes to git-send-email.perl +3 −0
diff --git a/git-send-email.perl b/git-send-email.perl
index 3564419..6adb669 100755
--- a/git-send-email.perl
+++ b/git-send-email.perl
@@ -773,6 +773,9 @@ X-Mailer: git-send-email $gitversion
 					$smtp = Net::SMTP::SSL->start_SSL($smtp)
 						or die "STARTTLS failed! ".$smtp->message;
 					$smtp_encryption = '';
+					# Send EHLO again to receive fresh
+					# supported commands
+					$smtp->hello();
 				} else {
 					die "Server does not support STARTTLS! ".$smtp->message;
 				}
-- 
1.5.6.2.224.g26efb.dirty
Junio C Hamano· Jul 9, 2008, 22:51 UTC · re: Robert Shearman · lore

Re: [PATCH] git-send-email: Fix authenticating on some servers when using TLS.

Robert Shearman <robertshearman@gmail.com> writes:
> Send hello after a successful STARTTLS command to refresh the list of extensions. These may be different to what is returned over a clear connection (for example the AUTH command may be accepted over a secure connection, but not over a clear connection).
>
> Furthermore, this behaviour is recommended by RFC 2487 (http://www.ietf.org/rfc/rfc2487.txt).
Very true.  Thanks.

← back to recent threads