Hi!
I'm wondering if the root cause of #1953 was incorrectly diagnosed, or if maybe Azure have changed their API again. If i peg my azurerm provider version to = 1.15 (i.e. the latest release before the "fix" in #1960 was merged) I can apply the following template without problems:
provider "azurerm" {
version = "= 1.15"
}
resource "azurerm_azuread_application" "ad_app" {
name = "tomas-testing"
}
In other words, I cannot reproduce the original issue (#1953).
However, as of #1960 and version 1.16, no URLs on the application can be http, which disallows something like this:
resource "azurerm_azuread_application" "ad_app" {
name = "tomas-testing"
reply_urls = ["http://localhost:8080"]
}
Again, this template applies successfully with 1.15 but fails on 1.16. This effectively makes impossible to e.g. set reply urls for AzureAD applications that enable local testing of OAuth flows (without configuring your local development environment to run https).
Is there any chance #1960 could be reverted for the next release, to re-enable this configuration?
PS. This template, touching all three types of URLs and setting them to HTTP-schemed values, also works without problems on 1.15:
provider "azurerm" {
version = "= 1.15"
}
resource "azurerm_azuread_application" "ad_app" {
name = "tomas-testing"
homepage = "http://tomas-testar.com"
identifier_uris = [
"http://tomas-testar",
]
reply_urls = ["http://localhost:8080"]
}
Hi @tomasaschan,
I've been able to verify that http is allowed again. I'm not sure why it wasn't for a while but ๐คทโโ๏ธ
I've opened #2320 to revert the change ๐
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!
Most helpful comment
Hi @tomasaschan,
I've been able to verify that http is allowed again. I'm not sure why it wasn't for a while but ๐คทโโ๏ธ
I've opened #2320 to revert the change ๐