Add new connection strategies and mark strategy as required #6
Conversation
New values (`apple`, `oidc`, `line`) are from the documentation: https://auth0.com/docs/api/management/v2#!/Connections/post_connections Additionally, `strategy` has been marked as required as the API states it is a mandatory parameter.
ghost
added
|
I just came here to see if this was under development! +1 would love this functionality. Thank you for contributing this! |
|
I'm not sure what is going on with the |
|
Hi Dan, thank you for submitting this! I'll give it a test and merge it in.
Indeed, there is a warning mechanism we can use. We just need to write it ourselves cause the terraform sdk doesn't come with one as far as I know. It's probably a good idea to do that, so we can support new connection types as they become available. |
|
@alexkappa thanks for merging this! Any chance we could cut a new release? We'd love to take advantage of this feature |
|
I asked hashicorp for a new release but they haven't gotten around to it yet. I will ping again. Meanwhile you could use the v0.5.2 from the original repository. |
New values (
apple,oidc,line) are from the documentation. I left the string list in the code in the same non-alphabetical order as the upstream documentation so it is easier to see what might change in the future.https://auth0.com/docs/api/management/v2#!/Connections/post_connections
Additionally,
strategyhas been marked as required as the API statesit is a mandatory parameter.
Open question- does it make sense to validate these since the list can continue to be updated in the future? Or is there a validator that will warn (but not reject) unknown but possibly valid values?
Community Note