allow reading oauth2 permissions from aad application

[janschumann]

Adds the oauth2Permissions manifest attribute to the exported attributes of a azuread_application resource and data resource.

https://docs.microsoft.com/en-us/azure/active-directory/develop/reference-app-manifest

[dominik-lekse]

@janschumann Thanks for doing the "boring" stuff. I am looking forward for this being merged into the provider soon.

[katbyte]

Thanks for the enhancement @janschumann,

I've left a few comments inline

[jfcoz]

@katbyte, is the review ok now ?

[katbyte]

Thanks for the revision @janschumann, aside from one minor comment this LGTM 👍 hope you don't mind but i am going to push a change to resolve the one outstanding issue so i can merge this today 🙂

[katbyte]

As this block is only exported/computed this can be removed

Suggested change

Computed: true,

[bluemalkin]

Hi, is there an ETA for a release with this fix please ?

[dbourcet]

I am curious about the syntax to use. If I have an application like so :

resource "azuread_application" "server" {
    name      = "server"
    reply_urls = ["http://server"]

what is the syntax to access the id of an oauth2_permissions block?
${azuread_application.server.oauth2_permissions[0]["id"] ?
${azuread_application.server.oauth2_permissions["id"] ?
I can't find how to use it.

[dbourcet]

I manage to do it this way: ${lookup(azuread_application.server.oauth2_permissions[0], "id")}.

[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 👉 hashibot-feedback@hashicorp.com. Thanks!