Skip to content

Commit 1b4fd5d

Browse files
committed
feat: Add some logs for oidc logout
1 parent 0c0aeaf commit 1b4fd5d

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

web/oidc/oidc.go

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -533,12 +533,14 @@ func Logout(c echo.Context) error {
533533
contextName := c.Param("context")
534534
conf, err := getGenericConfig(contextName)
535535
if err != nil {
536+
logger.WithNamespace("oidc").Errorf("Error on getGenericConfig for logout: %s", err)
536537
return c.JSON(http.StatusBadRequest, echo.Map{
537538
"error": "No OpenID Connect is configured",
538539
})
539540
}
540541

541542
if conf.IDTokenKeyURL == "" {
543+
logger.WithNamespace("oidc").Errorf("IDTokenKeyURL is not configured for logout")
542544
return c.JSON(http.StatusBadRequest, echo.Map{
543545
"error": "Cannot get the keys",
544546
"error_description": "id_token_jwk_url is not configured",
@@ -547,6 +549,7 @@ func Logout(c echo.Context) error {
547549

548550
keys, err := GetIDTokenKeys(conf.IDTokenKeyURL)
549551
if err != nil {
552+
logger.WithNamespace("oidc").Errorf("Error on GetIDTokenKeys for logout: %s", err)
550553
return c.JSON(http.StatusBadRequest, echo.Map{
551554
"error": "Cannot get the keys",
552555
"error_description": err,
@@ -583,6 +586,7 @@ func Logout(c echo.Context) error {
583586
}
584587
inst, err = findInstanceBySub(sub, contextName)
585588
if err != nil {
589+
logger.WithNamespace("oidc").Errorf("Error on findInstanceBySub for logout: %s", err)
586590
return c.JSON(http.StatusBadRequest, echo.Map{
587591
"error": "internal server error",
588592
"error_description": err,
@@ -599,6 +603,7 @@ func Logout(c echo.Context) error {
599603
domain = buildDomain(domain, conf)
600604
instance, err := lifecycle.GetInstance(domain)
601605
if err != nil {
606+
logger.WithNamespace("oidc").Errorf("Error on lifecycle.GetInstance for logout: %s", err)
602607
return c.JSON(http.StatusBadRequest, echo.Map{
603608
"error": "internal server error",
604609
"error_description": err,

0 commit comments

Comments
 (0)