Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Identity-based authentication for event_grid_output function-app decorator is not implemented in Python. #1501

Closed
luukk opened this issue May 23, 2024 · 2 comments · Fixed by Azure/azure-functions-python-library#229
Assignees
Labels

Comments

@luukk
Copy link

luukk commented May 23, 2024

According to the Python documentation for Event Grid output bindings, it is possible to connect to Event Grid with Identity-based authentication using the v2 Azure-functions model (https://learn.microsoft.com/en-us/azure/azure-functions/functions-bindings-event-grid-output?tabs=python-v2%2Cisolated-process%2Cnodejs-v4%2Cextensionv3&pivots=programming-language-python#identity-based-authentication). As explained in this section, you should "define the Connection property in the binding", suggesting that, and correct me if I interpreted this wrong, you should change the @app.event_grid_output to:
@app.event_grid_output( arg_name="outputEvent", connection=<CONNECTION_NAME_PREFIX>__topicEndpointUri).

However, using azure-functions with version 1.19.0", @app.event_grid_output() does not implement the connection parameter, causing an error with the message: "missing 2 required positional arguments: 'topic_endpoint_uri' and 'topic_key_setting'".

Is does seem that identity-based authentication is implemented and works for other input/output bindings, giving me the idea that the documentation describes a feature that has not been implemented.

@luukk luukk added the bug label May 23, 2024
@bhagyshricompany bhagyshricompany transferred this issue from Azure/Azure-Functions May 24, 2024
@bhagyshricompany bhagyshricompany self-assigned this Jun 25, 2024
@bhagyshricompany
Copy link

Hi @luukk please share the repro steps,fun name ,timestamp etc.

@gavin-aguiar
Copy link
Contributor

@luukk This bug has been fixed and will be available in the next azure functions release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants