From 9f44a00c6371d472346d2d277e9392d42c5af8aa Mon Sep 17 00:00:00 2001 From: Jan Dittberner Date: Sun, 19 May 2024 16:05:26 +0200 Subject: [PATCH] Fix linter warnings --- internal/handlers/login.go | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/internal/handlers/login.go b/internal/handlers/login.go index 473c1c7..875d6ee 100644 --- a/internal/handlers/login.go +++ b/internal/handlers/login.go @@ -127,6 +127,7 @@ func (h *LoginHandler) handleGet( for _, email := range certEmails { if *subject == email { subjectInCert = true + break } } @@ -275,13 +276,20 @@ func (h *LoginHandler) rejectLogin( w.WriteHeader(http.StatusFound) } -func (h *LoginHandler) rejectLoginMissingSubject(w http.ResponseWriter, r *http.Request, challenge string, localizer *i18n.Localizer, subject string) { +func (h *LoginHandler) rejectLoginMissingSubject( + w http.ResponseWriter, r *http.Request, challenge string, localizer *i18n.Localizer, subject string, +) { rejectRequest := client.NewRejectOAuth2RequestWithDefaults() - rejectRequest.SetErrorDescription(h.trans.LookupMessage("LoginDeniedSubjectMissing", map[string]interface{}{"Subject": subject}, localizer)) + rejectRequest.SetErrorDescription(h.trans.LookupMessage( + "LoginDeniedSubjectMissing", map[string]interface{}{"Subject": subject}, localizer), + ) rejectRequest.SetErrorHint(h.trans.LookupMessage("HintChooseDifferentClientCertificate", nil, localizer)) rejectRequest.SetStatusCode(http.StatusForbidden) - rejectLoginRequest, response, err := h.adminClient.RejectOAuth2LoginRequest(r.Context()).LoginChallenge(challenge).RejectOAuth2Request(*rejectRequest).Execute() + rejectLoginRequest, response, err := h.adminClient.RejectOAuth2LoginRequest( + r.Context()).LoginChallenge(challenge).RejectOAuth2Request( + *rejectRequest, + ).Execute() if err != nil { h.logger.Error("error sending reject login request", "error", err) http.Error(w, http.StatusText(http.StatusInternalServerError), http.StatusInternalServerError)