Skip to content

Commit

Permalink
Fix MFA detection with custom APP_BASE
Browse files Browse the repository at this point in the history
  • Loading branch information
speatzle committed Aug 9, 2024
1 parent e13f484 commit 79ccd2e
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion api/api.go
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ import (
"encoding/json"
"fmt"
"net/http"
"strings"
)

// APIResponse is the Struct representation of a Json Response
Expand Down Expand Up @@ -62,7 +63,7 @@ start:
if res.Header.Status == "success" {
return r, &res, nil
} else if res.Header.Status == "error" {
if res.Header.Code == 403 && res.Header.URL == "/mfa/verify/error.json" {
if res.Header.Code == 403 && strings.HasSuffix(res.Header.URL, "/mfa/verify/error.json") {
if !firstTime {
// if we are here this probably means that the MFA callback is broken, to prevent a infinite loop lets error here
return r, &res, fmt.Errorf("Got MFA challenge twice in a row, is your MFA Callback broken? Bailing to prevent loop...:")
Expand Down

0 comments on commit 79ccd2e

Please sign in to comment.