alexkappa / terraform-provider-auth0

ARCHIVED Auth0 Terraform Provider. This project is now being maintained at: https://github.com/auth0/terraform-provider-auth0
https://registry.terraform.io/providers/auth0/auth0/latest/docs
Mozilla Public License 2.0
320 stars 149 forks source link

Support for Custom SMS Gateways #416

Closed grese closed 2 years ago

grese commented 3 years ago

Description

I'd like to request support for gateway_url, provider, gateway_authentication, and forward_request_info fields in the auth0_connection resource. This would be used to replace Twilio with a custom SMS provider. The changes required are summarized below.

The full setup of a custom SMS gateway for passwordless is described in this Auth0 post.

New or Affected Resource(s)

Potential Terraform Configuration

resource "auth0_connection" "custom_sms_connection" {
  name     = "sms"
  strategy = "sms"
  ...
  options {
    provider    = "sms_gateway"
    gateway_url = "https://test.com/sms-gateway"
    gateway_authentication {
      method                = "bearer"
      subject               = "test.us.auth0.com:sms"
      audience              = "https://test.com/sms-gateway"
      secret                = "<My Shared Secret>"
      secret_base64_encoded = false
    }
    forward_request_info = true
    ...
  }
}

References

Community Note

grese commented 3 years ago

Just opened PR #417, and go-auth0/auth0 #230 which adds the options described in this PR to the auth0_connection resource.

tristan-curve commented 3 years ago

Just opened PR #417, and go-auth0/auth0 #230 which adds the options described in this PR to the auth0_connection resource.

what needs to happen to get this PR merged?

grese commented 3 years ago

@tristan-curve - Essentially, go-auth0/auth0 #230 just needs to be merged before #417. It looks to me like both of the PRs still need review by @alexkappa. If there are any requested changes during the review process, I'll just have to update the PRs.

sergiught commented 2 years ago

Closing this as this will now be available in the next release as https://github.com/alexkappa/terraform-provider-auth0/pull/417 got merged.

Thanks a lot for reporting this and opening the PR @grese !