Hey guys, I've been trying out the new sms support...
# talk-kratos
Hey guys, I've been trying out the new sms support, but I'm still getting config errors:
Copy code
kratos_1           | The configuration contains values or keys which are invalid:
kratos_1           | courier: map[sms:map[enabled:true from:+##########request_config:map[auth:map[config:map[password:*** user:sean.westfall@gmail.com] type:basic_auth] header:map[Content-Type:application/x-www-form-urlencoded] method:POST url:<https://api.twilio.com/2010-04-01/Accounts/***/Messages.json]]> smtp:map[connection_uri:<smtps://postmaster>@***:***@smtp.mailgun.org:465/?skip_ssl_verify=true from_address:no-reply@***]]
kratos_1           |     ^-- additionalProperties "sms" not allowed
and what my current config looks like for courier:
Copy code
    from_address: no-reply@***
    connection_uri: <smtps://postmaster>@***:***@smtp.mailgun.org:465/?skip_ssl_verify=true
    from: '+##########'
      url: <https://api.twilio.com/2010-04-01/Accounts/***/Messages.json>
      method: POST
        'Content-Type': 'application/x-www-form-urlencoded'
        type: basic_auth
          user: ***
          password: ***
    enabled: true
Hello, I’ve seen your message in the PR thread. Tried to recheck but wasn’t able to reproduce
Can you share the version of kratos that you are using and full config files?
Hey @User so here is my full config: https://pastebin.com/fXxqN8WA
thanks for working on the sms pr, also
the version I am using, well I don't know how to look up the version number, but I compile and create a docker image on my own (with the command
make docker
, that's how I've been doing it, and that's also how I tested the original pr).
so, what version did I compile, this is the print out from
git log
Copy code
commit 65004c11203712818016d478be3905e0c1fc8cdb (HEAD -> master, origin/master, origin/HEAD)
Author: Anirudh Oppiliappan <x@icyphox.sh>
Date:   Thu Feb 24 15:58:10 2022 +0530

    chore: use live mailchimp list ids (#2254)

commit 8c58e94707122a9b50873ca1acaa32659b5b8416
Author: aeneasr <3372410+aeneasr@users.noreply.github.com>
Date:   Wed Feb 23 14:00:54 2022 +0100

    fix: lower-case jsonnet context for sms

commit 799fa106cd0fed33afbe76903911df9292d49bf6
Author: aeneasr <3372410+aeneasr@users.noreply.github.com>
Date:   Wed Feb 23 13:55:02 2022 +0100

    test: add tests for new sms options
Hey @clean-glass-46254 were you able to solve this?
No, I am still trying to figure out how to use the sms feature @magnificent-energy-493