search

canonical / kratos-operator

Charmed Ory Kratos
https://charmhub.io/kratos
Apache License 2.0
4 stars 2 forks source link

Providers from the configmap are not parsed correctly #252

Closed nsklikas closed 2 months ago

nsklikas commented 2 months ago

Bug Description

Kratos does not parse the configmap created from the admin UI correctly. The cm looks like:

apiVersion: v1
data:
  idps.yaml: '[{"id":"9f097a05-7313-4dea-b9f1-fda4283414c5","provider":"github","label":"","client_id":"Ov23liItoyqEnJOkZlbo","client_secret":"18e81e7f9f72259c78697284318f580f64c28096","issuer_url":"","auth_url":"","token_url":"","microsoft_tenant":"","subject_source":"userinfo","apple_team_id":"","apple_private_key_id":"","apple_private_key":"","scope":["user:email"],"mapper_url":"file:///etc/config/kratos/okta_schema.jsonnet","requested_claims":null},{"id":"937a0b42-7222-4fb6-b2e3-284876314059","provider":"github","label":"","client_id":"Ov23liItoyqEnJOkZlbo","client_secret":"18e81e7f9f72259c78697284318f580f64c28096","issuer_url":"","auth_url":"","token_url":"","microsoft_tenant":"","subject_source":"userinfo","apple_team_id":"","apple_private_key_id":"","apple_private_key":"","scope":["user:email"],"mapper_url":"file:///etc/config/kratos/default.jsonnet","requested_claims":null}]'
kind: ConfigMap

The problem is that the kratos charm does not convert the contents of the cm from a string to a list (see https://github.com/canonical/kratos-operator/blob/main/src/charm.py#L730)

syncronize-issues-to-jira[bot] commented 2 months ago

Thank you for reporting us your feedback!

The internal ticket has been created: https://warthogs.atlassian.net/browse/IAM-1019.

This message was autogenerated