Hi 👋 some feedback regarding 🐛 Ory Kratos: it seems like its still needed to use a fake URI as a placeholder, if one decides to use

http

as

delivery_strategy

. Or to say it with other words: > smtp.connection_uri is still required even though delivery_strategy is set to http see the issues here and here. Improvement idea: provide a fix in Ory Kratos and/or the related helm charts to get rid of having placeholder URIs (and related secrets) for a configuration, that is never used. Kratos should be able to handle this 👍.

Thanks for the report Matthias, will look into it!

What version are you on?

kratos v1.1.0

see: https://github.com/ory/kratos/issues/3559#issuecomment-2007547584

Thanks, that’s right. I think it was fixed in the PR you mentioned, but must’ve gotten lost somewhere. Will be fixed in the next release.

thx a lot!

I managed to get a workaround for this issue using the older style of config for courier. Have only tested locally though and we are not running this in production:

courier:
  smtp:
    connection_uri: <smtps://whatever>
    from_address: <mailto:hello@careapp.com.au|hello@careapp.com.au>
    from_name: CareApp
  sms:
    enabled: true
    request_config:
      url: <https://api.twilio.com/2010-04-01/Accounts/AXXXXXXXXXXXXXX/Messages.json> # Adjust your account ID
      method: POST
      body: <base64://ZnVuY3Rpb24oY3R4KSB7CiAgVG86IGN0eC5yZWNpcGllbnQsCiAgQm9keTogY3R>4LmJvZHksCn0= # see below
      headers:
        Content-Type: application/x-www-form-urlencoded # defaults to application/json
      auth:
        type: basic_auth # or api_key
        config:
          user: AXXXXXXX # adjust your credentials
          password: XXXX # adjust your credentials

What is the issue your facing exactly @dazzling-napkin-4938? The original issue just meant that you need to supply some value for the connection_uri, but can still use the http based courier. It’s just a config validation issue.

Ah sorry I misread Mathias's post. I have run into this issue with Kratos 1.1.0, which is one of the issues preventing us from pushing ahead with sms based 2FA: https://github.com/ory/kratos/pull/3795 It's been fixed but not released yet. The config I posted above seems to be a functioning workaround though. Sorry for derailing this thread, I misread the issue