https://www.ory.sh/ logo
Powered by
c

clean-glass-46254

03/03/2022, 8:46 AM
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
courier:
  smtp:
    from_address: no-reply@***
    connection_uri: <smtps://postmaster>@***:***@smtp.mailgun.org:465/?skip_ssl_verify=true
  sms:
    from: '+##########'
    request_config:
      url: <https://api.twilio.com/2010-04-01/Accounts/***/Messages.json>
      method: POST
      header:
        'Content-Type': 'application/x-www-form-urlencoded'
      auth:
        type: basic_auth
        config:
          user: ***
          password: ***
    enabled: true
w

white-yak-22226

03/03/2022, 9:01 AM
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?
c

clean-glass-46254

03/06/2022, 10:37 PM
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
m

magnificent-energy-493

05/31/2022, 2:50 PM
Hey @clean-glass-46254 were you able to solve this?
c

clean-glass-46254

06/09/2022, 9:17 PM
No, I am still trying to figure out how to use the sms feature @magnificent-energy-493
10 Views
Powered by