Skip to content

Commit

Permalink
nit
Browse files Browse the repository at this point in the history
Signed-off-by: pmahindrakar-oss <[email protected]>
  • Loading branch information
pmahindrakar-oss committed Feb 6, 2024
1 parent 021a580 commit c6fb0ae
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions flyteadmin/auth/handlers.go
Original file line number Diff line number Diff line change
Expand Up @@ -158,8 +158,7 @@ func GetLoginHandler(ctx context.Context, authCtx interfaces.AuthenticationConte

idpUrl, err := url.Parse(urlString)
if err != nil {
errString := fmt.Sprintf("failed to parse url %q: %v", urlString, err)
logger.Error(ctx, errString)
logger.Errorf(ctx, "failed to parse url %q: %v", urlString, err)
writer.WriteHeader(http.StatusInternalServerError)
}

Check warning on line 163 in flyteadmin/auth/handlers.go

View check run for this annotation

Codecov / codecov/patch

flyteadmin/auth/handlers.go#L161-L163

Added lines #L161 - L163 were not covered by tests

Expand Down

0 comments on commit c6fb0ae

Please sign in to comment.