From 91fa3b30b098a1d06a2d6bf6d414e1f536dc6ea0 Mon Sep 17 00:00:00 2001
From: Shivaram Lingamneni <slingamn@cs.stanford.edu>
Date: Wed, 17 Jun 2020 03:08:38 -0400
Subject: [PATCH] fix the corresponding bug for certfp authentication

---
 irc/accounts.go | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/irc/accounts.go b/irc/accounts.go
index f34acde1..f5ceb757 100644
--- a/irc/accounts.go
+++ b/irc/accounts.go
@@ -1407,9 +1407,7 @@ func (am *AccountManager) AuthenticateByCertFP(client *Client, certfp, authzid s
 			AuthScriptInput{Certfp: certfp, IP: client.IP().String()})
 		if err != nil {
 			am.server.logger.Error("internal", "failed shell auth invocation", err.Error())
-			return err
-		}
-		if output.Success && output.AccountName != "" {
+		} else if output.Success && output.AccountName != "" {
 			clientAccount, err = am.loadWithAutocreation(output.AccountName, config.Accounts.AuthScript.Autocreate)
 			return
 		}