Add primary & secondary connection string to 'azurerm_iothub_dps_shared_access_policy'

[jackbatzner]

Add primary_connection_string and secondary_connection_string to azurerm_iothub_dps_shared_access_policy

--- PASS: TestAccAzureRMIotHubDpsSharedAccessPolicy_writeWithoutRead (151.97s)
--- PASS: TestAccAzureRMIotHubDpsSharedAccessPolicy_enrollmentReadWithoutRegistration (166.87s)
--- PASS: TestAccAzureRMIotHubDpsSharedAccessPolicy_enrollmentWriteWithoutOthers (205.74s)
--- PASS: TestAccAzureRMIotHubDpsSharedAccessPolicy_basic (273.52s)

[katbyte]

Thanks for this PR @Brunhil, aside from needing some nil checks this is looking great.

[jackbatzner]

Would it be easier if I created a new PR with the provider reorganization changes or pulled the changes in after the PR is approved @katbyte ?

[katbyte]

@Brunhil, Either or would work. Just one comment i've left inline that once addressed this'll be good to merge

[jackbatzner]

@Brunhil, Either or would work. Just one comment i've left inline that once addressed this'll be good to merge

End result for PR is ugly, but rebase is done :). Let me know your thoughts!

[katbyte]

Thanks for the revisions @Brunhil! LGTM now 👍

[ghost]

This has been released in version 1.40.0 of the provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. As an example:

provider "azurerm" {
    version = "~> 1.40.0"
}
# ... other configuration ...

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 [email protected]. Thanks!